smiklosovic commented on code in PR #4596:
URL: https://github.com/apache/cassandra/pull/4596#discussion_r2749181677
##########
src/java/org/apache/cassandra/db/guardrails/Guardrails.java:
##########
@@ -613,6 +613,15 @@ public final class Guardrails implements GuardrailsMBean
format("The keyspace %s has a replication factor of %s,
above the %s threshold of %s.",
what, value, isWarning ? "warning" : "failure",
threshold));
+ /**
+ * Guardrail on mis-prepared statements.
+ */
+ public static final EnableFlag MispreparedStatementsEnabled =
Review Comment:
start the guardrail name with lower-case as others are. Also, why is this
(to me correctly) called "mispreparedStatementsEnabled" but the config
introduces use_misprepare_statements_enabled ? Just align it, no?
--
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: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
