cshannon commented on PR #4254: URL: https://github.com/apache/accumulo/pull/4254#issuecomment-1951406170
@keith-turner - I think this is good to go now. I updated my test today to check and verify that the next compaction to run after writing more data to trigger a second SYSTEM compaction is a USER compaction and not the SYSTEM compaction that is triggered from the data write (my previous version yesterday only ensured the USER compaction eventually ran but did not guarantee it was the next one). Well, when I did this my test started to fail and was showing another SYSTEM compaction kept running. I started debugging and noticed that every time the check was done inside `CompactionCoordinator` to see if there were pending requested user compactions it was false even after it was set to true by the fate operation. I couldn't figure out why and finally found it in `TabletManagementIterator`...I forgot to add the new marker column to the list of columns to fetch, once I did that it started to read it and things now work as expected where the SYSTEM compaction gets blocked by the pending USER compaction when the test is run and if you change the fate operation to no longer set the marker then the test fails because Accumulo runs another SYSTEM compaction instead of the USER compaction. -- 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]
