Github user cestella commented on a diff in the pull request:
https://github.com/apache/incubator-metron/pull/79#discussion_r60250470
--- Diff: metron-streaming/Metron-DataLoads/pom.xml ---
@@ -167,56 +225,14 @@
</resources>
<plugins>
<plugin>
- <!-- Separates the unit tests from the integration tests.
-->
<groupId>org.apache.maven.plugins</groupId>
<artifactId>maven-surefire-plugin</artifactId>
- <version>2.12.4</version>
<configuration>
- <!-- Skip the default running of this plug-in (or
everything is run twice...see below) -->
- <argLine>-Xmx2048m -XX:MaxPermSize=256m
-Djava.util.logging.config.file=logging.properties</argLine>
+ <argLine>-Xmx2048m -XX:MaxPermSize=256m
-XX:-UseSplitVerifier</argLine>
<skip>true</skip>
- <!-- Show 100% of the lines from the stack trace
(doesn't work) -->
<trimStackTrace>false</trimStackTrace>
-
</configuration>
- <executions>
--- End diff --
So, I don't mind this, but this change will make the "integration tests"
run as part of the unit test cycle and people have gotten touchy in the past
due to the length that it adds to mvn package. I'm generally for it as it
ensures we get the full suite, but I wanted to make sure it was intended and
we're ok with it in general.
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---
