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

Removed cassandra-consistency from all the configs #2898

Merged

Conversation

anish531213
Copy link
Contributor

No description provided.

@coveralls
Copy link

coveralls commented Dec 10, 2019

Coverage Status

Coverage decreased (-0.02%) to 67.927% when pulling 5cddf3f on anish531213:aanish/remove-cassandra-consistency-config into 61d1a8a on uber:master.

Copy link
Contributor

@wxing1292 wxing1292 left a comment

Choose a reason for hiding this comment

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

also remove the corresponding doc in here https://github.com/uber/cadence/blob/a76b9574976dff34fd738997a3e57e4e869ad97d/docs/persistence.md

look for consistency: "QUORUM" -- Default consistence, MUST be QUORUM for production (optional)

@anish531213 anish531213 merged commit 9ca09cc into uber:master Dec 11, 2019
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

Successfully merging this pull request may close these issues.

3 participants