flink-issues mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From aljoscha <...@git.apache.org>
Subject [GitHub] flink pull request #5230: [FLINK-8345] Add iterator of keyed state on broadc...
Date Tue, 02 Jan 2018 13:17:59 GMT
Github user aljoscha commented on a diff in the pull request:

    https://github.com/apache/flink/pull/5230#discussion_r159222137
  
    --- Diff: flink-runtime/src/test/java/org/apache/flink/runtime/state/SerializationProxiesTest.java
---
    @@ -287,16 +339,60 @@ public void testOperatorStateMetaInfoReadSerializerFailureResilience()
throws Ex
     		PowerMockito.whenNew(TypeSerializerSerializationUtil.TypeSerializerSerializationProxy.class).withAnyArguments().thenReturn(mockProxy);
     
     		try (ByteArrayInputStreamWithPos in = new ByteArrayInputStreamWithPos(serialized))
{
    +			final DataInputViewStreamWrapper inWrapper = new DataInputViewStreamWrapper(in);
    +
     			metaInfo = OperatorBackendStateMetaInfoSnapshotReaderWriters
    -				.getReaderForVersion(OperatorBackendSerializationProxy.VERSION, Thread.currentThread().getContextClassLoader())
    --- End diff --
    
    This doesn't test reading the broadcast state, does it?


---

Mime
View raw message