ifesdjeen commented on code in PR #3711:
URL: https://github.com/apache/cassandra/pull/3711#discussion_r1863369072


##########
src/java/org/apache/cassandra/db/compaction/CompactionIterator.java:
##########
@@ -1240,7 +1239,7 @@ private static boolean isAccordTable(ColumnFamilyStore 
cfs, String name)
 
     private static boolean isAccordJournal(ColumnFamilyStore cfs)
     {
-        return isAccordTable(cfs, AccordKeyspace.JOURNAL);
+        return 
cfs.getKeyspaceName().equals(SchemaConstants.ACCORD_KEYSPACE_NAME) && 
cfs.name.startsWith(AccordKeyspace.JOURNAL);

Review Comment:
   We are relying on the fact that each journal in burn test is using its own 
table, so we separate their compactions, too. I will add a comment that 
elaborates on this.



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