[jira] [Comment Edited] (FLINK-5186) Move Row and RowTypeInfo into Flink core

["Fabian Hueske (JIRA)" <ji...@apache.org>]

    [ 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
(v6.3.4#6332)