Skip to content
This repository has been archived by the owner on Sep 4, 2020. It is now read-only.

Added kryo setter method in Builder class #82

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

anoop44
Copy link

@anoop44 anoop44 commented May 24, 2016

No description provided.

@nhachicha
Copy link
Owner

Hi @anoop44
Thanks for the PR,
this is good but obviously this will override any previously registered Kryo instance (example)

we need to find a good trade-off between the two operations

@anoop44
Copy link
Author

anoop44 commented May 25, 2016

Hi @nhachicha

Thats Right. My concern was there was no option to set Defaualt Serializer in Builder. So I thought instead of just adding a setter method for DefaultSeriazer, it would be better to set kryo itself. So that user can have much customized kryo for them.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants