maedhroz commented on a change in pull request #1180:
URL: https://github.com/apache/cassandra/pull/1180#discussion_r700387998



##########
File path: src/java/org/apache/cassandra/config/Config.java
##########
@@ -349,8 +349,15 @@
     public volatile int tombstone_warn_threshold = 1000;
     public volatile int tombstone_failure_threshold = 100000;
 
+    // coordiantor reads
     public volatile long client_large_read_warn_threshold_kb = 0;
     public volatile long client_large_read_abort_threshold_kb = 0;

Review comment:
       It looks like `client_large_read_warn_threshold_kb` and 
`client_large_read_abort_threshold_kb` are only in trunk. Perhaps we could just 
prefix them with `coordinator_` here and be rid of the comments?




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