flink-issues mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Fabian Hueske (JIRA)" <j...@apache.org>
Subject [jira] [Comment Edited] (FLINK-5186) Move Row and RowTypeInfo into Flink core
Date Mon, 05 Dec 2016 08:04:58 GMT

    [ https://issues.apache.org/jira/browse/FLINK-5186?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15721535#comment-15721535

Fabian Hueske edited comment on FLINK-5186 at 12/5/16 8:04 AM:

+1 let's do this before Flink 1.2

Row as initially only used internally in flink-table.
However, more modules such as flink-jdbc use this class now. However, a dependency against
flink-table (and therefore Apache Calcite) can cause problems (see FLINK-5227).

was (Author: fhueske):
+1 let's do this before Flink 1.2

> Move Row and RowTypeInfo into Flink core
> ----------------------------------------
>                 Key: FLINK-5186
>                 URL: https://issues.apache.org/jira/browse/FLINK-5186
>             Project: Flink
>          Issue Type: Improvement
>          Components: Core, Table API & SQL
>            Reporter: Anton Solovev
>            Priority: Critical
>             Fix For: 1.2.0
> Those classes needs to be reworked before. They should not depend on Scala-related things.
RowTypeInfo should not inherit from CaseClassTypeInfo, the current solution with the dummy
field names is a hacky solution anyway. Row should not inherit from Scala classes. Also Row
should support unlimited count of fields

This message was sent by Atlassian JIRA

View raw message