Repository: calcite
Updated Branches:
  refs/heads/master 0c0f61e96 -> d74027c98


[CALCITE-2519] Silence ERROR logs from CalciteException, SqlValidatorException 
during tests

Use threadCount=1/perCoreThreadCount=false to disable parallel test execution 
at Travis

Previous option -Dsurefire.parallel= made JUnit to log just CalciteSuite 
details,
thus the test looked like a stuck one

fixes #816


Project: http://git-wip-us.apache.org/repos/asf/calcite/repo
Commit: http://git-wip-us.apache.org/repos/asf/calcite/commit/d74027c9
Tree: http://git-wip-us.apache.org/repos/asf/calcite/tree/d74027c9
Diff: http://git-wip-us.apache.org/repos/asf/calcite/diff/d74027c9

Branch: refs/heads/master
Commit: d74027c989ac8b5cd6cd38c04bbabf9ab0e844d6
Parents: 0c0f61e
Author: Vladimir Sitnikov <[email protected]>
Authored: Fri Aug 31 22:57:40 2018 +0300
Committer: Vladimir Sitnikov <[email protected]>
Committed: Sat Sep 1 04:42:21 2018 +0300

----------------------------------------------------------------------
 .travis.yml                              | 2 +-
 core/src/test/resources/log4j.properties | 5 +++++
 pom.xml                                  | 6 ++++--
 3 files changed, 10 insertions(+), 3 deletions(-)
----------------------------------------------------------------------


http://git-wip-us.apache.org/repos/asf/calcite/blob/d74027c9/.travis.yml
----------------------------------------------------------------------
diff --git a/.travis.yml b/.travis.yml
index 45a9430..411e116 100644
--- a/.travis.yml
+++ b/.travis.yml
@@ -44,7 +44,7 @@ install:
 script:
   # Print surefire output to the console instead of files
   - unset _JAVA_OPTIONS
-  - $DOCKERRUN $IMAGE mvn -Dsurefire.useFile=false -Dsurefire.parallel= 
-Djavax.net.ssl.trustStorePassword=changeit test
+  - $DOCKERRUN $IMAGE mvn -Dsurefire.useFile=false -Dsurefire.threadCount=1 
-Dsurefire.perCoreThreadCount=false -Djavax.net.ssl.trustStorePassword=changeit 
test
 git:
   depth: 10000
 sudo: required

http://git-wip-us.apache.org/repos/asf/calcite/blob/d74027c9/core/src/test/resources/log4j.properties
----------------------------------------------------------------------
diff --git a/core/src/test/resources/log4j.properties 
b/core/src/test/resources/log4j.properties
index 834e2db..02b2ef7 100644
--- a/core/src/test/resources/log4j.properties
+++ b/core/src/test/resources/log4j.properties
@@ -16,6 +16,11 @@
 # Root logger is configured at INFO and is sent to A1
 log4j.rootLogger=INFO, A1
 
+# [CALCITE-2519] Silence ERROR logs from CalciteException, 
SqlValidatorException during tests
+log4j.logger.org.apache.calcite.runtime.CalciteException=FATAL
+log4j.logger.org.apache.calcite.sql.validate.SqlValidatorException=FATAL
+log4j.logger.org.apache.calcite.plan.RexImplicationChecker=ERROR
+
 # A1 goes to the console
 log4j.appender.A1=org.apache.log4j.ConsoleAppender
 

http://git-wip-us.apache.org/repos/asf/calcite/blob/d74027c9/pom.xml
----------------------------------------------------------------------
diff --git a/pom.xml b/pom.xml
index d7dfdc0..51a029a 100644
--- a/pom.xml
+++ b/pom.xml
@@ -47,6 +47,8 @@ limitations under the License.
 
   <properties>
     <surefire.parallel>both</surefire.parallel>
+    <surefire.threadCount>1</surefire.threadCount>
+    <surefire.perCoreThreadCount>true</surefire.perCoreThreadCount>
 
     <project.build.sourceEncoding>UTF-8</project.build.sourceEncoding>
     <top.dir>${project.basedir}</top.dir>
@@ -925,8 +927,8 @@ limitations under the License.
           <groupId>org.apache.maven.plugins</groupId>
           <artifactId>maven-surefire-plugin</artifactId>
           <configuration>
-            <threadCount>1</threadCount>
-            <perCoreThreadCount>true</perCoreThreadCount>
+            <threadCount>${surefire.threadCount}</threadCount>
+            
<perCoreThreadCount>${surefire.perCoreThreadCount}</perCoreThreadCount>
             <parallel>${surefire.parallel}</parallel>
             <forkMode>once</forkMode>
             <systemProperties>

Reply via email to