juripetersen commented on code in PR #638:
URL: https://github.com/apache/incubator-wayang/pull/638#discussion_r2601488279
##########
wayang-platforms/wayang-flink/src/main/java/org/apache/wayang/flink/compiler/WayangFileOutputFormat.java:
##########
@@ -125,6 +125,7 @@ public static void
initDefaultsFromConfiguration(Configuration configuration) {
private transient DataOutputViewStreamWrapper outView;
+ private ObjectFileSerializationMode serializationMode =
ObjectFileSerializationMode.LEGACY_JAVA_SERIALIZATION;
Review Comment:
I think it depends on the consequences for the user. If they have to
implement further things to make sure their data types implement Serializable,
I agree. If this is a change without any overhead for the user, fixing a
potential security threat, then it should be the new default.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
