You are viewing a plain text version of this content. The canonical link for it is here.
Posted to issues@flink.apache.org by "ASF GitHub Bot (JIRA)" <ji...@apache.org> on 2017/02/22 09:34:44 UTC

[jira] [Commented] (FLINK-5863) Unify the serialization of queryable list states in different backends

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

ASF GitHub Bot commented on FLINK-5863:
---------------------------------------

GitHub user shixiaogang opened a pull request:

    https://github.com/apache/flink/pull/3387

    [FLINK-5863][queryable state] Add the serialization of list states in KvStateRequestSerializer

    1. Add `serializeList()` in `KvStateRequestSerialization`
    2. Modify the unit tests of `KvStateRequestSerialization`, without the access to protected methods.
    3. Move `KvStateRequestSerializationRocksDBTest` from package `org.apache.flink.test.query` to `org.apache.flink.contrib.streaming.state`.

You can merge this pull request into a Git repository by running:

    $ git pull https://github.com/alibaba/flink flink-5863

Alternatively you can review and apply these changes as the patch at:

    https://github.com/apache/flink/pull/3387.patch

To close this pull request, make a commit to your master/trunk branch
with (at least) the following in the commit message:

    This closes #3387
    
----
commit 5fc51b5f233ec62e143c1938b11c677b2260575b
Author: xiaogang.sxg <xi...@alibaba-inc.com>
Date:   2017-02-22T03:56:24Z

    Add the serialization of list states in KvStateRequestSerializer

----


> Unify the serialization of queryable list states in different backends
> ----------------------------------------------------------------------
>
>                 Key: FLINK-5863
>                 URL: https://issues.apache.org/jira/browse/FLINK-5863
>             Project: Flink
>          Issue Type: Improvement
>          Components: Queryable State
>    Affects Versions: 1.3.0
>            Reporter: Xiaogang Shi
>            Assignee: Xiaogang Shi
>            Priority: Minor
>
> Now the deserialization of list states is implemented in {{KvStateRequestSerializer}}. The serialization however is implemented individually in different backends. 
> We should provide a method in {{KvStateRequestSerializer}} to remove the redundant code.



--
This message was sent by Atlassian JIRA
(v6.3.15#6346)