nwangtw opened a new pull request #3283: Set to Kryo serializer when registerKryoSerialization is called URL: https://github.com/apache/incubator-heron/pull/3283 In Storm, users just need to register class serializer. Kryo is the default serializer. However in Heron, the default is Java serializer which doesn't have the functionality. I think it is more consistent to set to KryoSerializer when the registration function is called. Another option is to set Kryo serializer as the default one. The concern is that some users might be affected by the implicit change. I am totally open to discussion.
---------------------------------------------------------------- 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. For queries about this service, please contact Infrastructure at: [email protected] With regards, Apache Git Services
