jmark99 opened a new pull request #420: ACCUMULO-4697 tablet server could kick 
off too many idle compactions
URL: https://github.com/apache/accumulo/pull/420
 
 
   ACCUMULO-4697 tablet server could kick off too many idle compactions
   
   Tablet.initiateMajorCompaction() always returns false, but we check the 
return value when accounting for the number of idle major compactions started 
in TabletServer.MajorCompactor.run() and continue to kick off idle compactions 
on every tablet.
   
   DefaultCompactionStrategy doesn't do anything special for idle compactions, 
so this will only be an issue with a custom CompactionStrategy.
   
   My guess would be the best thing to do would be to get rid of idle 
compactions and get rid of the return value on initiateMajorCompaction().
   
   This update follows the reporters suggestion and removes the idle compaction 
code and refactors the  initiateMajorCompaction() method to return void.
   
   This ticket references Jira ticket  
[ACCUMULO-4697](https://issues.apache.org/jira/browse/ACCUMULO-4697)
   

----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


With regards,
Apache Git Services

Reply via email to