-
Notifications
You must be signed in to change notification settings - Fork 30
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
eventually-sled: new backend based on sled db #122
Comments
Hey @pwoolcoc, For a new backend implementation, you need to implement the following traits:
For now, let's only consider the first two traits. For point 2. it seems pretty easy to implement using the reactive semantics offered by sled. For point 1. we need to figure out how to store the events and how to stream them back. When storing and streaming, we need double indexing: one on a single aggregate level (aggregate id), and one on aggregate type level (aggregate type id). Also for point 1., we need to figure out a way to use optimistic concurrency in the write operations, in order to prevent concurrent writes scenarios. What do you think? @pwoolcoc |
@ar3s3ru thanks! sorry I didn't respond to this thread earlier, but I'm definitely still interested in doing this and I appreciate the pointers and the reference! |
Ok, initial thoughts, in no particular order:
|
@ar3s3ru quick question: I'm hoping to have a POC of this soon, should I open a PR and try to get it in-tree or should I just keep in my own repo? |
Hey @pwoolcoc 👋 Feel free to open a PR to have the CI tests run 😄 |
I brought up a possible backend implementation using the sled database, and danilo suggested I open a tracking issue to discuss it. (I'll post more specific thoughts later, just wanted to get this open for now so I didn't forget)
The text was updated successfully, but these errors were encountered: