[GitHub] [cassandra] adelapena commented on a change in pull request #1444: CASSANDRA-17187 trunk: Guardrail for SELECT IN terms and their cartesian product

[GitBox <gi...@apache.org>]

adelapena commented on a change in pull request #1444:
URL: https://github.com/apache/cassandra/pull/1444#discussion_r820937974



##########
File path: src/java/org/apache/cassandra/db/guardrails/GuardrailsMBean.java
##########
@@ -221,6 +221,17 @@
      */
     void setReadBeforeWriteListOperationsEnabled(boolean enabled);
 
+    /**
+     * @return The threshold to warn when the number of partition keys in a select statement greater than threshold.
+     * -1 means disabled.
+     */
+    int getPartitionKeysInSelectWarnThreshold();

Review comment:
       Those aren't new, I have just changed the order to have first the getters, then the setters, like in the rest of the properties. I find it easier to read if all methods follow a consistent order.




-- 
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-unsubscribe@cassandra.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org



---------------------------------------------------------------------
To unsubscribe, e-mail: pr-unsubscribe@cassandra.apache.org
For additional commands, e-mail: pr-help@cassandra.apache.org