Github user aljoscha commented on a diff in the pull request:
https://github.com/apache/flink/pull/5567#discussion_r171214451
--- Diff:
flink-core/src/main/java/org/apache/flink/util/InstantiationUtil.java ---
@@ -113,18 +113,61 @@ public ClassLoaderObjectInputStream(InputStream in,
ClassLoader classLoader) thr
*
* <p>This can be removed once 1.2 is no longer supported.
*/
- private static Set<String> scalaSerializerClassnames = new HashSet<>();
+ private final static Set<String> scalaSerializerClassnames = new
HashSet<>();
static {
scalaSerializerClassnames.add("org.apache.flink.api.scala.typeutils.TraversableSerializer");
scalaSerializerClassnames.add("org.apache.flink.api.scala.typeutils.CaseClassSerializer");
scalaSerializerClassnames.add("org.apache.flink.api.scala.typeutils.EitherSerializer");
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");
--- End diff --
nvm, @twalthr pointed out that it was in there twice
---
