cshannon opened a new pull request, #3923: URL: https://github.com/apache/accumulo/pull/3923
This updates tests in `TabletManagementIteratorIT` to verify that the iterator properly detects tablets needing attention if WALs exist on tablets that are part of a merge operation. It also will test that delete does not care if there are WALs. This PR only updates the tests in TabletManagementIteratorIT and doesn't try and do anything with TGW because after the refactoring in #3904 both the iterator and TGW share the same code for checking the goal state (where the WAL checks are done) However, another possible follow on test improvement would be to create a test for the `TabletGoalState` class where all the logic was refactored to inside of compute() to test the different cases for computing the resulting state. This could likely just be a normal unit test with mocking and not an IT. -- 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]
