DomGarguilo opened a new pull request, #5370: URL: https://github.com/apache/accumulo/pull/5370
This PR makes several improvements to some ITs. The most notable improvement is converting several ITs to use `SharedMiniClusterBase` in order to speed them up. I totaled the test times before and after these changes and saw over 13 mins to complete before and 6 mins to complete with these changes. The most significant changes were in `AmpleConditionalWriterIT`. A big decrease in runtime was observed. There were also a bunch of test cases that were creating an `AccumuloClient` and not using it so I remove that from those test cases. Additionally, I moved one of the test cases (`testSuspendMarker()`) into its own test class (`SuspendMarkerIT`) since it seemed like it needed to use its own MAC. I took the chance to clean things up a bit when copying it into its own file but did not change any of the test functionality. There were a few places were I replaces the string `"table.custom.assignment.group"` with `TABLE_ASSIGNMENT_GROUP_PROPERTY` in a few files. I simplified some logic in `TabletResourceGroupBalanceIT` to use `stopTabletServerGroup()` which must not have existed when the test was written. -- 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]
