cshannon commented on issue #4710:
URL: https://github.com/apache/accumulo/issues/4710#issuecomment-2282242812

   > I can work on creating a test for 
`CompactionCoordinator.compactionFailedForLevel` so that we can close out this 
issue as `CompactionDriver.cleanupTabletMetadata` will be opened as a separate 
issue. I'll probably try and do something with TestAmple first as that has the 
interceptor to support the unknown status but FlakyAmpleManager is also an 
option depending on what is easiest when I take a look.
   
   I opened up #4797 and I went with using FlakyAmpleManager after all as that 
was by far the simplest way to test after having tried to use TestAmple. There 
were already good tests for cancellation that would exercise the failed 
compaction code so using Flaky ample exercises the rejection handler inside 
`CompactionCoordinator.compactionFailedForLevel`


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

Reply via email to