Timo Walther commented on FLINK-15038:

I think this makes sense. But it might cause problems until we remove all the 
\{{TypeInformation}}. Currently we might use the conversion class for the type 
info <-> data type conversion. If there are no problems +1 for this.

> Get field DataType of TableSchema should return DataType with default 
> conversion class
> --------------------------------------------------------------------------------------
>                 Key: FLINK-15038
>                 URL: https://issues.apache.org/jira/browse/FLINK-15038
>             Project: Flink
>          Issue Type: Bug
>          Components: Table SQL / API
>            Reporter: Jingsong Lee
>            Priority: Major
>             Fix For: 1.10.0
> Now, the user and planner construct TableSchema with various DataTypes, them 
> maybe have various conversion classes.
> For example, return DataType of Functions maybe with various TypeInformations 
> and lead to various conversion classes.
> TableSchema should only have logical informations instead of various 
> conversion classes.
> This ticket want to clean the field DataType in TableSchema and return a 
> consistent conversion classes to users.
> Modify:
> getFieldDataTypes/getFieldDataType/toRowDataType in TableSchema.
> Before return, we can bridgeTo default conversion class using 
> TypeConversions.fromLogicalToDataType(type.getLogicalType()).

This message was sent by Atlassian Jira

Reply via email to