flink-issues mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "ASF GitHub Bot (JIRA)" <j...@apache.org>
Subject [jira] [Commented] (FLINK-5790) Use list types when ListStateDescriptor extends StateDescriptor
Date Tue, 14 Feb 2017 11:03:41 GMT

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

ASF GitHub Bot commented on FLINK-5790:

Github user StephanEwen commented on a diff in the pull request:

    --- Diff: flink-core/src/main/java/org/apache/flink/api/common/typeutils/base/ArrayListSerializer.java
    @@ -108,9 +112,13 @@ public void copy(DataInputView source, DataOutputView target) throws
     	public boolean equals(Object obj) {
    +		if (obj.getClass() != getClass())  {
    +			System.out.println("other " + obj.getClass().getName() + ", this " + getClass().getName());
    --- End diff --
    This is probably left over code that should be removed.

> 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
> 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

View raw message