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

JAMES-3435 Fine grain LWT configuration for the Cassandra mailbox #350

Merged
merged 12 commits into from
Apr 8, 2021

Conversation

chibenwa
Copy link
Contributor

| mailbox.read.strong.consistency
| Optional. Boolean, defaults to true. Disabling should be considered experimental.
If enabled, regular consistency level is used for read transactions for mailbox. Not doing so might result
in stale reads as the system.paxos table will not be checked for latest updates. Better performance are expected
by turning it off. Note that reads performed as part of write transactions are always performed with a strong
consistency.

| message.read.strong.consistency
| Optional. Boolean, defaults to true. Disabling should be considered experimental.
If enabled, regular consistency level is used for read transactions for message. Not doing so might result
in stale reads as the system.paxos table will not be checked for latest updates. Better performance are expected
by turning it off. Note that reads performed as part of write transactions are always performed with a strong
consistency.

| message.write.strong.consistency.unsafe
| Optional. Boolean, defaults to true. Disabling should be considered experimental and unsafe.
If enabled, Lightweight transactions will no longer be used upon messages operation (table `imapUidTable`).
As message flags updates relies so far on a read-before-write model, it exposes yourself to data races leading to
potentially update loss. Better performance are expected
by turning it off. Reads performed as part of write transaction are also performed with a relaxed consistency.

@chibenwa
Copy link
Contributor Author

chibenwa commented Apr 1, 2021

Build was 🍏 only squashed fixups.

@chibenwa
Copy link
Contributor Author

chibenwa commented Apr 1, 2021

If there's some reviewers out there that would like to give some <3 to this work?

@jeantil
Copy link
Contributor

jeantil commented Apr 1, 2021

I gave it a cursory look but I haven't dug into mailbox code yet, so I have no idea what i'm looking at which is why I abstained to review further.

@chibenwa chibenwa merged commit 4619763 into apache:master Apr 8, 2021
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