[ https://issues.apache.org/jira/browse/FLINK-5790?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15867155#comment-15867155 ]
ASF GitHub Bot commented on FLINK-5790: --------------------------------------- Github user shixiaogang commented on the issue: https://github.com/apache/flink/pull/3305 @tillrohrmann Thanks for your review. Sorry for the reformatted code. It seems that my IDE will automatically reformat all the files I've edited. I will revert the reformatted code. > Use list types when ListStateDescriptor extends StateDescriptor > --------------------------------------------------------------- > > Key: FLINK-5790 > URL: https://issues.apache.org/jira/browse/FLINK-5790 > Project: Flink > Issue Type: Improvement > Reporter: Xiaogang Shi > Assignee: Xiaogang Shi > > Flink keeps the state serializer in {{StateDescriptor}}, but it's the > serializer of list elements that is put in {{ListStateDescriptor}}. The > implementation is a little confusing. Some backends need to construct the > state serializer with the element serializer by themselves. > We should use an {{ArrayListSerializer}}, which is composed of the serializer > of the element, in the {{ListStateDescriptor}}. It helps the backend to avoid > constructing the state serializer. > If a backend needs customized serialization of the state (e.g. > {{RocksDBStateBackend}}), it still can obtain the element serializer from the > {{ArrayListSerializer}}. -- This message was sent by Atlassian JIRA (v6.3.15#6346)