The Wayback Machine - https://web.archive.org/web/20201117185300/https://github.com/eventflow/EventFlow/pull/760
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

Make serialization scheme more generic #760

Open
wants to merge 9 commits into
base: develop
from

Conversation

@tedvanderveen
Copy link

@tedvanderveen tedvanderveen commented Jun 2, 2020

At the moment, quite a few implementation in the library are tightly coupled to JSON de/serialization.
For my current project, I need to use a more compact binary serialized scheme.
This PR makes event serialization scheme more generic, and allows for dropping in binary serialization.

This adds support for using for example Protobuf of BSON serialization of events.

tedvanderveen and others added 2 commits Jun 2, 2020
Merge from origin
@CLAassistant
Copy link

@CLAassistant CLAassistant commented Jun 2, 2020

CLA assistant check
All committers have signed the CLA.

tedvanderveen added 5 commits Jun 3, 2020
…set serialiaztion scheme (string)
Add support for serialization schemes with snapshot stores
Set generic constraint on TSerialized (parameter must implement IEnumerable, such as string and byte[])
@tedvanderveen
Copy link
Author

@tedvanderveen tedvanderveen commented Jun 3, 2020

Hi @rasmus, looks like this PR is ready for review (AppVeyor is still running at the moment).
There should be no changes to the library's existing public APIs.
Looking forward to your feedback!

@tedvanderveen
Copy link
Author

@tedvanderveen tedvanderveen commented Jun 3, 2020

@rasmus looks like appveyor timed out, can you restart the build please?

tedvanderveen
@tedvanderveen
Copy link
Author

@tedvanderveen tedvanderveen commented Jun 4, 2020

@rasmus looks like appveyor timed out, can you restart the build please?

@rasmus added a missing newline at end of a file, to trigger new build ;-)

@tedvanderveen
Copy link
Author

@tedvanderveen tedvanderveen commented Jun 4, 2020

@rasmus the appveyor build timed out again (after 1 hour). Does this happen more often? Can you see more details maybe on what causes it to time out?

@rasmus
Copy link
Member

@rasmus rasmus commented Jun 4, 2020

Normally it doesn't, I'll have a look this evening

@rasmus
Copy link
Member

@rasmus rasmus commented Jun 7, 2020

Usually its the RabbitMQ tests that die, this seems a bit odd.

@rasmus
Copy link
Member

@rasmus rasmus commented Jun 7, 2020

While I do like the cleanup, this change has several braking changes thus we'll have to wait until a major release.

While 0.x is technically a beta version, many rely in the current APIs and I try not to do any API breaking changes. I know that there are plenty companies out there that have implemented their own event store to better suit their infrastructure.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Linked issues

Successfully merging this pull request may close these issues.

None yet

3 participants
You can’t perform that action at this time.