You are viewing a plain text version of this content. The canonical link for it is here.
Posted to issues@flink.apache.org by "Congxian Qiu (JIRA)" <ji...@apache.org> on 2018/07/18 02:46:00 UTC
[jira] [Commented] (FLINK-6799) PojoSerializer does not check field
types when being deserialized
[ https://issues.apache.org/jira/browse/FLINK-6799?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16547311#comment-16547311 ]
Congxian Qiu commented on FLINK-6799:
-------------------------------------
Hi, [~tzulitai] is this issue still needed? If you do not have enough time, I could take it over, thank you.
> PojoSerializer does not check field types when being deserialized
> -----------------------------------------------------------------
>
> Key: FLINK-6799
> URL: https://issues.apache.org/jira/browse/FLINK-6799
> Project: Flink
> Issue Type: Bug
> Components: Type Serialization System
> Affects Versions: 1.1.5, 1.2.1, 1.3.0, 1.4.0
> Reporter: Till Rohrmann
> Assignee: Tzu-Li (Gordon) Tai
> Priority: Major
>
> The {{PojoSerializer}} stores internally the fields of a pojo as {{Field}} instances. These field instances are serialized via the {{FieldSerializer}}. The {{FieldSerializer}} simply writes out the name of the field. When being deserialized, the serializer reads the name and then looks up the field using the dynamically loaded class. If the underlying class changes, e.g. type of fields change, then this won't be noticed, because we don't check the field types. In order to catch errors early and to give meaningful error messages, I think we should add type checks for the reloaded fields.
--
This message was sent by Atlassian JIRA
(v7.6.3#76005)