-
Notifications
You must be signed in to change notification settings - Fork 3.6k
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
autorepair v2 framework #3306
base: trunk
Are you sure you want to change the base?
autorepair v2 framework #3306
Conversation
jaydeepkumar1984
commented
May 13, 2024
- AutoRepir v2 framework that covers both FR & IR
- An interface with a default implementation for TokenRange calculation
- Enable/Disable flag as a CQL Table property
93d7b83
to
154c790
Compare
ColumnFamilyStore columnFamilyStore = keyspace.getColumnFamilyStore(tableName); | ||
if (columnFamilyStore.metadata().params.disableAutomatedRepair) | ||
{ | ||
logger.info("Repair is disabled for keyspace {} for tables: {}", keyspaceName, tableName); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
logger.info("Repair is disabled for keyspace {} for tables: {}", keyspaceName, tableName); | |
logger.info("Repair is disabled for keyspace {} for table: {}", keyspaceName, tableName); |
|
||
public void setRepairMinIntervalInHours(RepairType repairType, int repairMinFrequencyInHours) | ||
{ | ||
repair_type_overrides.get(repairType).min_repair_interval_in_hours = repairMinFrequencyInHours; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
JFYI, there's a bug in these setters. If you provide a config in cassandra.yaml
such as the following
auto_repair:
repair_type_overrides:
full:
enabled: true
repair_type_overrides.get("incremental")
will be null and we'll get an NPE if we call a setter with incremental repair type
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Though this only affects nodetool functionality
e225d41
to
27dcbf2
Compare
27dcbf2
to
75798ac
Compare