himanshujindal commented on code in PR #4304: URL: https://github.com/apache/cassandra/pull/4304#discussion_r2274492668
########## src/java/org/apache/cassandra/repair/autorepair/AutoRepair.java: ########## @@ -165,6 +166,11 @@ public void repair(AutoRepairConfig.RepairType repairType) logger.debug("Auto-repair is disabled for repair type {}", repairType); return; } + if (!DatabaseDescriptor.getMixedMajorVersionRepairEnabled() && + Gossiper.instance.hasMultipleLiveMajorVersions()) { + logger.info("Auto-repair is disabled when nodes in the cluster have different major versions"); Review Comment: I am not seeing this class emit metrics. I'll be happy to introduce metrics to this class. Do you have an example of code where we do emit metric, so I can use that pattern to emit metrics? -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: pr-unsubscr...@cassandra.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org --------------------------------------------------------------------- To unsubscribe, e-mail: pr-unsubscr...@cassandra.apache.org For additional commands, e-mail: pr-h...@cassandra.apache.org