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

Added kryo setter method in Builder class#82

Open
anoop44 wants to merge 1 commit into
nhachicha:masterfrom
anoop44:feature/kryo_setter_in_builder
Open

Added kryo setter method in Builder class#82
anoop44 wants to merge 1 commit into
nhachicha:masterfrom
anoop44:feature/kryo_setter_in_builder

Conversation

@anoop44
Copy link
Copy Markdown

@anoop44 anoop44 commented May 24, 2016

No description provided.

@nhachicha
Copy link
Copy Markdown
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
Copy Markdown
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