Maxwell-Guo commented on code in PR #2287:
URL: https://github.com/apache/cassandra/pull/2287#discussion_r1243048046


##########
src/java/org/apache/cassandra/config/Config.java:
##########
@@ -1103,6 +1103,11 @@ public enum PaxosOnLinearizabilityViolation
     public volatile long min_tracked_partition_tombstone_count = 5000;
     public volatile boolean top_partitions_enabled = true;
 
+    /**
+     * Default compaction configuration, used if a table does not specify any.
+     */
+    public ParameterizedClass default_compaction = null;

Review Comment:
   > The flag specifies a default value for the compaction parameter. Could you 
elaborate on why it should be renamed?
   
   Actually, it feels like there's no need to change it 



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