DomGarguilo opened a new pull request, #4737: URL: https://github.com/apache/accumulo/pull/4737
Tried to improve things by refactoring the code to make things more readable. Also converted the longs to use the `NanoTime` object. The logic should be identical before and after this change. I did notice that the compactor idle check seems to include the same check that is happening in the code that reads it (`AbstractServer.idleProcessCheck()`) where it checks the `idleReportingPeriod` has elapsed since the last compaction has completed before returning true. Unless there is a reason this was added, it seems like that part could just be removed and the same check would take place upstream. -- 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]
