[GitHub] [incubator-heron] nwangtw opened a new pull request #3283: Set to Kryo serializer when registerKryoSerialization is called

[GitBox <gi...@apache.org>]

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:
users@infra.apache.org


With regards,
Apache Git Services