smiklosovic commented on code in PR #2961:
URL: https://github.com/apache/cassandra/pull/2961#discussion_r1418690820


##########
src/java/org/apache/cassandra/db/guardrails/Guardrails.java:
##########
@@ -477,6 +478,21 @@ public final class Guardrails implements GuardrailsMBean
                      format("The modification to table %s has a timestamp %s 
before the minimum allowable %s threshold %s",
                             what, value, isWarning ? "warning" : "failure", 
threshold));
 
+    public static final MaxThreshold 
maximumReferencedIndexesWithNonParitionRestrictedQuery =

Review Comment:
   I chose to go with guardrail approach even we are not checking guardrail as 
such (will be visible further in down in the patch). This has a nice property 
of being update-able via JMX in runtime which would be impossible with a static 
configuration property.



-- 
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]

Reply via email to