[GitHub] [accumulo] keith-turner commented on pull request #3059: fixes #3045 remove stale compactions from coordinator

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

keith-turner commented on PR #3059:
URL: https://github.com/apache/accumulo/pull/3059#issuecomment-1298595882

   > Looks good, but I wonder if we should put the cleanup in a background thread instead of on the request thread that the Monitor uses to get the data. I'm not sure that this is super-important, as I think the orphaned RUNNING entries happen in a small number of cases. But, if the Monitor is down or never used by the user, then no cleanup will happen.
   
   When I first started looking into this I was considering adding this code to the DeadCompactionDetector because it already scans the metadata table and runs periodically in the background.  I decided against this because I thought it may make the code harder to understand and would force these two task to run at the same cadence.  I think I will take another look at making it run in the background as a task or as part of the dead compaction detector.
   
   > Test seems to fail on JDK 17. Not sure what's going on with that.
   
   I saw that, going to look into it.  Seemed to be failing to initialize the class, wonder if thats because I called System.nanoTime to initialize an instance var and maybe JDK 17 does not like that.


-- 
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: notifications-unsubscribe@accumulo.apache.org

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