[GitHub] [cassandra] dcapwell commented on pull request #642: Cassandra 15861 lock trunk

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

dcapwell commented on pull request #642:
URL: https://github.com/apache/cassandra/pull/642#issuecomment-685069342


   > index summary redistribution and stats-mutation shouldn't happen together because both are running as part of compactions.
   
   You are right, I keep forgetting `org.apache.cassandra.db.lifecycle.Tracker#tryModify(java.lang.Iterable<org.apache.cassandra.io.sstable.format.SSTableReader>, org.apache.cassandra.db.compaction.OperationType)` marks sstables as compacting, so compactions will avoid compacting those files
   
   ```
   if (null == apply(permitCompacting(sstables), updateCompacting(emptySet(), sstables)))
               return null;
           return new LifecycleTransaction(this, operationType, sstables);
   ```


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

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