You are viewing a plain text version of this content. The canonical link for it is here.
Posted to issues@flink.apache.org by "Paul Lin (Jira)" <ji...@apache.org> on 2020/05/10 13:29:00 UTC

[jira] [Commented] (FLINK-17091) AvroRow(De)SerializationSchema doesn't support new Timestamp conversion classes

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

Paul Lin commented on FLINK-17091:
----------------------------------

[~jark] May I ask your opinion on this issue? There's some similar issues reported recently, like FLINK-17486 and FLINK-17478.

> AvroRow(De)SerializationSchema doesn't support new Timestamp conversion classes
> -------------------------------------------------------------------------------
>
>                 Key: FLINK-17091
>                 URL: https://issues.apache.org/jira/browse/FLINK-17091
>             Project: Flink
>          Issue Type: Bug
>          Components: Formats (JSON, Avro, Parquet, ORC, SequenceFile)
>    Affects Versions: 1.10.0
>            Reporter: Paul Lin
>            Priority: Major
>
> AvroRow(De)SerializationSchema doesn't know how to convert the new physical classes of Timestamp (eg. java.time.Date) to/from Avro's int/long based timestamp. Currently, when encountering objects of the new physical classes, AvroRow(De)SerializationSchema just ignores them and passes them to Avro's GenericDatumWriter/Reader, which leads to ClassCastException thrown by GenericDatumWriter/Reader. See [AvroRowSerializationSchema|https://github.com/apache/flink/blob/master/flink-formats/flink-avro/src/main/java/org/apache/flink/formats/avro/AvroRowSerializationSchema.java#L251].
> To fix this problem, we should support LocalTime/LocalDate/LocalDateTime conversion to long in AvroRowSerializationSchema, and support long conversion to LocalTime/LocalDate/LocalDateTime based on logical types(Types.LOCAL_TIME/Types.LOCAL_DATE/Types.LOCAL_DATE_TIME) in AvroRowDeserializationSchema.



--
This message was sent by Atlassian Jira
(v8.3.4#803005)