Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Turning off parquet logging in SparkFunSuite tests #29

Closed
tdanford opened this issue Dec 13, 2013 · 1 comment · Fixed by #30
Closed

Turning off parquet logging in SparkFunSuite tests #29

tdanford opened this issue Dec 13, 2013 · 1 comment · Fixed by #30

Comments

@tdanford
Copy link
Contributor

(I'm going to experiment with opening this as an issue, so that it's on our collective minds and maybe someone will remember how to track it down?)

If you're using the SparkFunSuite for Spark-based tests, with sparkTest, sparkBefore, and sparkAfter, there's a silenceSpark option that allows you to turn off Spark logging. However, if you're using adamSave as part of your spark test, even silenceSpark=true won't silence the parquet.hadoop INFO level logging coming out of (I presume) Parquet.

I tried adding "parquet" and "parquet.hadoop" to the hard-coded list of silenced packages in the SparkLogUtils, and that didn't work for me either.

If someone can figure out where the Parquet logging is enabled (my guess: a log4j.xml embedded in the Parquet dependency?) and whether it could be modified at runtime, that would take us a long way towards figuring out how to turn it off during tests. It's making the test outcome output nearly unreadable.

Actually, generalizing that list of hard-coded silenced packages (or allowing the user to add new packages to it "on the fly" from the test as an argument) might be another reasonable change in the future too.

@massie
Copy link
Member

massie commented Dec 13, 2013

ADAM has a utility function, e.g. ParquetLogger.hadoopLoggerLevel(Level.SEVERE). While Spark uses SFL4J and a log4j backend, Parquet uses Java Logging.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants