Skip to content

DB Crashes on startup if accord tables exist but accord is disabled #4197

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

Open
wants to merge 1 commit into
base: trunk
Choose a base branch
from

Conversation

pranavshenoy
Copy link
Contributor

Moved the check to validate. This ensures accord flag is checked only during table creation and not during replay.

patch by Pranav Shenoy; reviewed David by TBD for CASSANDRA-20600

Moved the check to validate. This ensures accord flag is checked only
during table creation and not during replay.

patch by Pranav Shenoy; reviewed David by TBD for CASSANDRA-20600
@dcapwell
Copy link
Contributor

My biggest question is what happens to the server once it starts up. If accord isn't enabled but the tables are marked as accord tables, then don't we have an outage for these tables?

Copy link
Contributor

@JeremiahDJordan JeremiahDJordan left a comment

Choose a reason for hiding this comment

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

LGTM. Creation will still fail when queries are sent in, but should not fail node startup.
I would ask someone in the #cassandra-accord channel to be another reviewer here.

@pranavshenoy
Copy link
Contributor Author

My biggest question is what happens to the server once it starts up. If accord isn't enabled but the tables are marked as accord tables, then don't we have an outage for these tables?

@dcapwell I had a chat with @JeremiahDJordan regarding this.
The expectation is that the accord queries would fail but startup would work.

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