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

[ADR] Against the use of Lightweight Transaction in Cassandra code base #271

Closed
wants to merge 4 commits into from

Conversation

chibenwa
Copy link
Contributor

@chibenwa chibenwa commented Dec 4, 2020

No description provided.

Co-authored-by: Rene Cordier <rene.cordier@gmail.com>
Copy link
Member

@rouazana rouazana left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Rely on event sourcing to maintain a projection of ACLs that do not rely on LWT or SERIAL consistency level.

LWT are required for eventSourcing. As event sourcing usage is limited to low-usage use cases, the performance
degradations are not an issue.

I'm probably missing your point, but for me these 2 sentences seems completely contradictory.

@rouazana
Copy link
Member

rouazana commented Dec 4, 2020

I'm probably missing your point, but for me these 2 sentences seems completely contradictory.

I think I got it reading your code. You assume that with event sourcing you use LWT only for writes and that you have a projection for reads.
This explanation is missing.
And then you are missing an explanation of why you cannot do that directly for every usage of LWT.
(note that I have nothing against event sourcing, and I think it's very valuable in particular for things like ACL, but your explanation is incomplete for me)

@chibenwa
Copy link
Contributor Author

Merged

I noticed clearly that related works requires more discussions and mentionned alternative ways to potentially generate monotic integers.

@chibenwa chibenwa closed this Dec 14, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
3 participants