Skip to content

Sritchie merge#2

Open
sunng87 wants to merge 8 commits intosritchie:masterfrom
sunng87:sritchie-merge
Open

Sritchie merge#2
sunng87 wants to merge 8 commits intosritchie:masterfrom
sunng87:sritchie-merge

Conversation

@sunng87
Copy link

@sunng87 sunng87 commented Jan 17, 2012

Hi sritchie,

In this pull request, I made two major changes on carbonite:

  • Use kryo's build-in ObjectBuffer as default buffer;
  • Improve Keyword serializer. We should try to avoid using read-string and prn as serializer.

@sritchie
Copy link
Owner

Hey, I'm really happy about this pull request, but i'm not actually the base fork. Can you submit this pull request to the revelytix branch?

@sunng87
Copy link
Author

sunng87 commented Jan 19, 2012

Actually I have sent pull request to revelytix. But it seems they are not very active on this project. And I thought you might have token over the project ...
Anyway, I will send them a reminder about that pull request.

Thank you.

@sunng87 sunng87 closed this Jan 19, 2012
@sritchie
Copy link
Owner

@sunng87, I'll write to the revelytix guys and see what I can do! We'll get this merged for you one way or another. I'll keep this open for now until we figure out how it'll get resolved.

@sritchie sritchie reopened this Jan 19, 2012
@yn
Copy link

yn commented May 12, 2014

Did this happen?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants