Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Replace transit #1

Open
lgrapenthin opened this issue Sep 25, 2019 · 2 comments
Open

Replace transit #1

lgrapenthin opened this issue Sep 25, 2019 · 2 comments
Assignees

Comments

@lgrapenthin
Copy link

transit-clj README states:

NOTE: Transit is intended primarily as a wire protocol for transferring data between applications. If storing Transit data durably, readers and writers are expected to use the same version of Transit and you are responsible for migrating/transforming/re-storing that data when and if the transit format changes.

@gws
Copy link
Owner

gws commented Nov 2, 2019

@lgrapenthin Thank you for the report!

I don't have a good excuse for that decision since I was aware of that even back when I first wrote this library. At the time I figured my session data was short-lived and so it wouldn't be a big deal, but that only worked for my use case and there's always going to be a point that a user will need to "cut over" with potential data loss if there are breaking changes to the Transit protocol.

If I were to see about a change to the underlying storage format, I would probably use nippy, do you have other suggestions?

@gws gws self-assigned this Nov 2, 2019
@lgrapenthin
Copy link
Author

I'd prefer Fressian because its used in Datomic and I prefer the authors. But I have not used nippy and the proposed speed advantage surely looks promising.

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

No branches or pull requests

2 participants