flink-issues mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Stephan Ewen (JIRA)" <j...@apache.org>
Subject [jira] [Commented] (FLINK-2963) Dependence on SerializationUtils#deserialize() should be avoided
Date Thu, 05 Nov 2015 18:44:27 GMT

    [ https://issues.apache.org/jira/browse/FLINK-2963?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14992216#comment-14992216
] 

Stephan Ewen commented on FLINK-2963:
-------------------------------------

I would like to avoid it as well. We have our own deserialization method in {{InstantiationUtil}}.
No need to use anything external.
/cc [~aljoscha]

> Dependence on SerializationUtils#deserialize() should be avoided
> ----------------------------------------------------------------
>
>                 Key: FLINK-2963
>                 URL: https://issues.apache.org/jira/browse/FLINK-2963
>             Project: Flink
>          Issue Type: Bug
>            Reporter: Ted Yu
>            Priority: Minor
>
> There is a problem with `SerializationUtils` from Apache Commons
> Lang. Here is an open issue where the class will throw a
> `ClassNotFoundException` even if the class is in the classpath in a
> multiple-classloader environment:
> https://issues.apache.org/jira/browse/LANG-1049
> {code}
> 			state = (HashMap<String, Serializable>) SerializationUtils.deserialize(bais);
> ./flink-streaming-java/src/main/java/org/apache/flink/streaming/runtime/operators/windowing/NonKeyedWindowOperator.java
> 			state = (HashMap<String, Serializable>) SerializationUtils.deserialize(bais);
> ./flink-streaming-java/src/main/java/org/apache/flink/streaming/runtime/operators/windowing/WindowOperator.java
> 		return SerializationUtils.deserialize(message);
> ./flink-streaming-java/src/main/java/org/apache/flink/streaming/util/serialization/JavaDefaultStringSchema.java
> 		T copied = SerializationUtils.deserialize(SerializationUtils
> ./flink-streaming-java/src/test/java/org/apache/flink/streaming/util/MockOutput.java
> {code}
> We should move away from SerializationUtils.deserialize()



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)

Mime
View raw message