Skip to content

Commit

Permalink
[FLINK-11073] [state backends, tests] Activate ignored testKeyedListS…
Browse files Browse the repository at this point in the history
…tateSerializerReconfiguration test in StateBackendMigrationTestBase

This test was previously ignored due to 2 missing changes:
- ListSerializerSnapshot was not respecting serializer reconfiguration
  (fixed by FLINK-11073), and
- RocksDBListState did not use the correct registered state serializer
  in backend's state meta info (fixed by FLINK-11287)

With those fixes in, the test can now be activated and expected to pass.
  • Loading branch information
tzulitai committed Jan 9, 2019
1 parent e5ed8c8 commit cad0509
Showing 1 changed file with 0 additions and 2 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -40,7 +40,6 @@
import org.apache.flink.util.TestLogger;

import org.junit.Assert;
import org.junit.Ignore;
import org.junit.Rule;
import org.junit.Test;
import org.junit.rules.TemporaryFolder;
Expand Down Expand Up @@ -192,7 +191,6 @@ public void testKeyedListStateMigration() throws Exception {
}

@Test
@Ignore("This currently doesn't pass because the ListSerializer doesn't respect the reconfigured case, yet.")
public void testKeyedListStateSerializerReconfiguration() throws Exception {
final String stateName = "test-name";

Expand Down

0 comments on commit cad0509

Please sign in to comment.