Skip to content
This repository has been archived by the owner on Feb 16, 2024. It is now read-only.

[BAHIR-84] Suppress Parquet-MR log messages from build log #33

Conversation

ckadner
Copy link
Member

@ckadner ckadner commented Jan 12, 2017

BAHIR-84: Build log flooded with test log messages

Proposed Changes

Parquet-MR (1.7.0) uses Java Logging, not Log4j, so we ...

  • add a new properties file: sql-streaming-mqtt/src/test/resources/logging.properties
  • increase the logger threshold for org.apache.parquet.hadoop.* to SEVERE
  • add the the new logging.properties to the maven-surefire-plugin and scalatest-maven-plugin via the java.util.logging.config.file system property

How was this change tested?

  • ran mvn clean package
  • quick test with mvn clean test -pl sql-streaming-mqtt -q

Result

$ mvn clean test -pl sql-streaming-mqtt -q

-------------------------------------------------------
 T E S T S
-------------------------------------------------------
Java HotSpot(TM) 64-Bit Server VM warning: ignoring option MaxPermSize=512m; support was removed in 8.0

Results :

Tests run: 0, Failures: 0, Errors: 0, Skipped: 0

Java HotSpot(TM) 64-Bit Server VM warning: ignoring option MaxPermSize=512m; support was removed in 8.0
Discovery starting.
SLF4J: Class path contains multiple SLF4J bindings.
SLF4J: Found binding in [jar:file:/Users/ckadner/.m2/repository/org/slf4j/slf4j-log4j12/1.7.16/slf4j-log4j12-1.7.16.jar!/org/slf4j/impl/StaticLoggerBinder.class]
SLF4J: Found binding in [jar:file:/Users/ckadner/.m2/repository/org/apache/activemq/activemq-all/5.13.3/activemq-all-5.13.3.jar!/org/slf4j/impl/StaticLoggerBinder.class]
SLF4J: See http://www.slf4j.org/codes.html#multiple_bindings for an explanation.
SLF4J: Actual binding is of type [org.slf4j.impl.Log4jLoggerFactory]
SLF4J: Failed to load class "org.slf4j.impl.StaticLoggerBinder".
SLF4J: Defaulting to no-operation (NOP) logger implementation
SLF4J: See http://www.slf4j.org/codes.html#StaticLoggerBinder for further details.
Discovery completed in 323 milliseconds.
Run starting. Expected test count is: 7
BasicMQTTSourceSuite:
- basic usage
- Send and receive 100 messages.
- no server up
- params not provided.
- Recovering offset from the last processed offset. !!! IGNORED !!!
StressTestMQTTSource:
- Send and receive messages of size 250MB. !!! IGNORED !!!
LocalMessageStoreSuite:
- serialize and deserialize
- Store and retreive
- Max offset stored
MQTTStreamSourceSuite:
Run completed in 21 seconds, 346 milliseconds.
Total number of tests run: 7
Suites: completed 5, aborted 0
Tests: succeeded 7, failed 0, canceled 0, ignored 2, pending 0
All tests passed.

@ApacheBahir
Copy link

Build failed, see build log for details

@ApacheBahir
Copy link

Refer to this link for build results (access rights to CI server needed):
http://169.45.79.58:8080/job/Apache%20Bahir%20-%20Pull%20Request%20Builder/29/

@ApacheBahir
Copy link

Build successful

@ApacheBahir
Copy link

Refer to this link for build results (access rights to CI server needed):
http://169.45.79.58:8080/job/Apache%20Bahir%20-%20Pull%20Request%20Builder/30/

@lresende
Copy link
Member

@ckadner Could you please reduce the size of the pr title or commit title before you try to merge.

@ckadner ckadner changed the title [BAHIR-84] suppress "...INFO: org.apache.parquet.hadoop..." log messages from build log [BAHIR-84] Suppress Parquet-MR log messages from build log Jan 18, 2017
@asfgit asfgit closed this in bce9cd1 Jan 20, 2017
@ckadner ckadner deleted the BAHIR-84_silence-parquet-hadoop-log-messages branch January 20, 2017 03:18
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants