[GitHub] [accumulo] ctubbsii commented on pull request #2015: Pom changes to allow reuseForks to be set to true

[GitBox <gi...@apache.org>]

ctubbsii commented on pull request #2015:
URL: https://github.com/apache/accumulo/pull/2015#issuecomment-819282264


   In order to run the problematic tests with `reuseForks=true` set for the others, you'll need to exclude these classes from the main configuration of `maven-surefire-plugin` with `<excludes>` (this can be done in core/pom.xml and server/tserver/pom.xml, rather than the root pom.xml, where you've made the current changes), and then you will need to have an additional execution of the surefire plugin, as in the following (not complete):
   
   ```xml
   <build>
     <plugins>
       <plugin>
         <artifactId>maven-surefire-plugin</artifactId>
         <executions>
           <execution>
             <id>no-reuse-forks-tests</id>
             <goals>
               <goal>test</goal>
             </goals>
             <configuration>
               <reuseForks>false</reuseForks>
               <includes />
             </configuration>
       </plugin>
     </plugins>
   </build>
   ```
   
   What I'm not sure about is whether the plugin's second execution will clobber any test output from the previous (default, built-in) execution.


-- 
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:
users@infra.apache.org