Github user zentol commented on a diff in the pull request:
https://github.com/apache/flink/pull/5567#discussion_r170215340
--- Diff:
flink-core/src/main/java/org/apache/flink/util/InstantiationUtil.java ---
@@ -121,10 +121,53 @@ public ClassLoaderObjectInputStream(InputStream in,
ClassLoader classLoader) thr
scalaSerializerClassnames.add("org.apache.flink.api.scala.typeutils.EnumValueSerializer");
scalaSerializerClassnames.add("org.apache.flink.api.scala.typeutils.OptionSerializer");
scalaSerializerClassnames.add("org.apache.flink.api.scala.typeutils.TrySerializer");
-
scalaSerializerClassnames.add("org.apache.flink.api.scala.typeutils.EitherSerializer");
scalaSerializerClassnames.add("org.apache.flink.api.scala.typeutils.UnitSerializer");
}
+ /**
+ * The serialVersionUID might change between Scala versions and since
those classes are
+ * part of the tuple serializer config snapshots we need to ignore them.
+ *
+ * See also FLINK-8451.
+ */
+ private static Set<String> scalaTypes = new HashSet<>();
--- End diff --
make final?
---
