ctubbsii commented on pull request #2007: URL: https://github.com/apache/accumulo/pull/2007#issuecomment-818460037
@Manno15 Okay, so, I did a partial revert, changing the default value for reuseForks to false for surefire, and removing the includes/excludes, just to get things working again, so we can get other PRs merged. If some tests can be renamed to the `*IT` pattern, and placed in a different (sub)package, then that's one option. Another option would be to run an extra execution of maven-surefire-plugin for just those tests, with reuseForks specifically turned off (similar to how minicluster/pom.xml forces forkCount to be 1). However, it may be okay just to leave this configurable for now, and set the default value to true later, after further improvements. I'm not sure how much work any of these options are. -- 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. For queries about this service, please contact Infrastructure at: [email protected]
