[GitHub] flink pull request #6308: [FLINK-9799] Generalize and unify state meta info ...

2018-07-12 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/flink/pull/6308 ---

[GitHub] flink pull request #6308: [FLINK-9799] Generalize and unify state meta info ...

2018-07-12 Thread StefanRRichter
Github user StefanRRichter commented on a diff in the pull request: https://github.com/apache/flink/pull/6308#discussion_r202047581 --- Diff: flink-runtime/src/main/java/org/apache/flink/runtime/state/KeyedBackendSerializationProxy.java --- @@ -98,8 +103,7 @@ public int

[GitHub] flink pull request #6308: [FLINK-9799] Generalize and unify state meta info ...

2018-07-12 Thread StefanRRichter
Github user StefanRRichter commented on a diff in the pull request: https://github.com/apache/flink/pull/6308#discussion_r202047403 --- Diff: flink-runtime/src/test/java/org/apache/flink/runtime/state/SerializationProxiesTest.java --- @@ -126,11 +131,11 @@ public void

[GitHub] flink pull request #6308: [FLINK-9799] Generalize and unify state meta info ...

2018-07-12 Thread StefanRRichter
Github user StefanRRichter commented on a diff in the pull request: https://github.com/apache/flink/pull/6308#discussion_r202041046 --- Diff: flink-runtime/src/main/java/org/apache/flink/runtime/state/metainfo/StateMetaInfoSnapshotReadersWriters.java --- @@ -0,0 +1,412 @@ +/*

[GitHub] flink pull request #6308: [FLINK-9799] Generalize and unify state meta info ...

2018-07-12 Thread azagrebin
Github user azagrebin commented on a diff in the pull request: https://github.com/apache/flink/pull/6308#discussion_r201798531 --- Diff: flink-runtime/src/main/java/org/apache/flink/runtime/state/OperatorBackendSerializationProxy.java --- @@ -62,62 +67,62 @@ public int

[GitHub] flink pull request #6308: [FLINK-9799] Generalize and unify state meta info ...

2018-07-12 Thread azagrebin
Github user azagrebin commented on a diff in the pull request: https://github.com/apache/flink/pull/6308#discussion_r202035286 --- Diff: flink-runtime/src/test/java/org/apache/flink/runtime/state/SerializationProxiesTest.java --- @@ -126,11 +131,11 @@ public void

[GitHub] flink pull request #6308: [FLINK-9799] Generalize and unify state meta info ...

2018-07-12 Thread azagrebin
Github user azagrebin commented on a diff in the pull request: https://github.com/apache/flink/pull/6308#discussion_r202033136 --- Diff: flink-runtime/src/main/java/org/apache/flink/runtime/state/metainfo/StateMetaInfoSnapshotReadersWriters.java --- @@ -0,0 +1,412 @@ +/*

[GitHub] flink pull request #6308: [FLINK-9799] Generalize and unify state meta info ...

2018-07-11 Thread sihuazhou
Github user sihuazhou commented on a diff in the pull request: https://github.com/apache/flink/pull/6308#discussion_r201778299 --- Diff: flink-runtime/src/main/java/org/apache/flink/runtime/state/KeyedBackendSerializationProxy.java --- @@ -98,8 +103,7 @@ public int getVersion() {

[GitHub] flink pull request #6308: [FLINK-9799] Generalize and unify state meta info ...

2018-07-11 Thread StefanRRichter
GitHub user StefanRRichter opened a pull request: https://github.com/apache/flink/pull/6308 [FLINK-9799] Generalize and unify state meta info snapshot ## What is the purpose of the change This PR generalizes and unifies the de/serialization of state meta information in