From cad0509a752be57eb995a95ccebf00012e443ed5 Mon Sep 17 00:00:00 2001
From: "Tzu-Li (Gordon) Tai" <tzulitai@apache.org>
Date: Tue, 8 Jan 2019 17:41:38 +0100
Subject: [PATCH] [FLINK-11073] [state backends, tests] Activate ignored
 testKeyedListStateSerializerReconfiguration 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.
---
 .../flink/runtime/state/StateBackendMigrationTestBase.java      | 2 --
 1 file changed, 2 deletions(-)

diff --git a/flink-runtime/src/test/java/org/apache/flink/runtime/state/StateBackendMigrationTestBase.java b/flink-runtime/src/test/java/org/apache/flink/runtime/state/StateBackendMigrationTestBase.java
index 8d85e74aca7..b636ed08bd2 100644
--- a/flink-runtime/src/test/java/org/apache/flink/runtime/state/StateBackendMigrationTestBase.java
+++ b/flink-runtime/src/test/java/org/apache/flink/runtime/state/StateBackendMigrationTestBase.java
@@ -40,7 +40,6 @@ import org.apache.flink.util.StateMigrationException;
 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;
@@ -192,7 +191,6 @@ public abstract class StateBackendMigrationTestBase<B extends AbstractStateBacke
 	}
 
 	@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";
 
-- 
GitLab