dcapwell commented on code in PR #4107:
URL: https://github.com/apache/cassandra/pull/4107#discussion_r2054842160


##########
src/java/org/apache/cassandra/db/compaction/LeveledCompactionStrategy.java:
##########
@@ -52,6 +52,7 @@ public class LeveledCompactionStrategy extends 
AbstractCompactionStrategy
     public static final String LEVEL_FANOUT_SIZE_OPTION = "fanout_size";
     public static final String SINGLE_SSTABLE_UPLEVEL_OPTION = 
"single_sstable_uplevel";
     public static final int DEFAULT_LEVEL_FANOUT_SIZE = 10;
+    public static final int DEFAULT_MAX_SSTABLE_SIZE_MIB = 160;

Review Comment:
   from https://github.com/apache/cassandra/pull/4105



##########
src/java/org/apache/cassandra/index/sai/disk/v1/MemtableIndexWriter.java:
##########
@@ -217,8 +217,8 @@ private void flushVectorIndex(long startTime, Stopwatch 
stopwatch) throws IOExce
 
     private void completeIndexFlush(long cellCount, long startTime, Stopwatch 
stopwatch) throws IOException
     {
-        // create a completion marker indicating that the index is complete 
and not-empty
-        ColumnCompletionMarkerUtil.create(indexDescriptor, indexIdentifier, 
false);
+        // create a completion marker indicating that the index is complete
+        ColumnCompletionMarkerUtil.create(indexDescriptor, indexIdentifier, 
cellCount == 0);

Review Comment:
   from https://github.com/apache/cassandra/pull/4105



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

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


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

Reply via email to