kevinrr888 opened a new pull request, #5579:
URL: https://github.com/apache/accumulo/pull/5579

   Some mini cluster ITs were missing a
   `SharedMiniClusterBase.stopMiniCluster()` in a JUnit AfterAll. Added to 
those that were missing.
   
   This has already been fixed in main by #5555
   
   Mini cluster needs to be started in a `BeforeAll` with 
`SharedMiniClusterBase.startMiniCluster()` or 
`SharedMiniClusterBase.startMiniClusterWithConfig(...)` and stopped in an 
`AfterAll` with `SharedMiniClusterBase.stopMiniCluster()`, so used:
   ```
   $ grep -rl --include *.java "startMiniCluster" . | xargs grep -L 
"stopMiniCluster"
   ```
   To identify tests not adhering to this


-- 
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: notifications-unsubscr...@accumulo.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org

Reply via email to