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

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

GitHub user StefanRRichter opened a pull request:

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

    [FLINK-5442] Ensure order of enum elements in StateDescriptor.Type

    This PR adds a test to ensure the order of elements in enum 
``StateDescriptor.Type`` which is important for the serialization format.

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

    $ git pull https://github.com/StefanRRichter/flink fix-descriptor-type-order

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

    https://github.com/apache/flink/pull/3091.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 #3091
    
----
commit e79b00a2a623dbd06329beee5c81086302821fae
Author: Stefan Richter <[email protected]>
Date:   2017-01-10T21:52:58Z

    [FLINK-5442] Ensure order of enum elements in StateDescriptor.Type through 
test

----


> Add test to fix ordinals of serialized enum StateDescriptor.Type
> ----------------------------------------------------------------
>
>                 Key: FLINK-5442
>                 URL: https://issues.apache.org/jira/browse/FLINK-5442
>             Project: Flink
>          Issue Type: Improvement
>          Components: State Backends, Checkpointing
>            Reporter: Stefan Richter
>            Assignee: Stefan Richter
>            Priority: Minor
>
> The order in which elements are contained in the enum 
> {{StateDescriptor.Type}} is important because the ordinal is used for custom 
> de/serialization. Changing this order would break the serialization format. 
> We should add a test that enforces the current order.



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)

Reply via email to