accumulo icon indicating copy to clipboard operation
accumulo copied to clipboard

Published 20 hours ago • verified publisher icon apache

Add tests to verify merge and delete ops will not proceed if WALs exist

Open cshannon opened this issue 1 year ago • 1 comments

This is a follow on for #3903 and #3904 to add tests to verify that the new checks for making sure that WALs don't exist during merge/delete operations work properly. It would be nice to update TabletManagementIteratorIT and also possibly test the changes in TabletGroupWatcher which may be easier after #3904

cshannon avatar Oct 30 '23 22:10 cshannon

I need to revisit this more but this issue may be a good candidate to use the new test API coming in #4415 . The new test API could be used to easily insert some fake WAL entries and then the merge and delete repos could be tested. This could also likely just be tested with a normal IT but should be a lot easier with the test API as we can specifically test just those FATE repos and the logic we care about. I still need to look more at TabletManagementIteratorIT to see about testing that, that could be split out into a follow on if we needed.

cshannon avatar Apr 12 '24 12:04 cshannon

Tests were added in #4415, so I am closing this. If more tests are needed it can be a follow on

cshannon avatar May 11 '24 13:05 cshannon