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

[SPARK-7261][Core] Change default log level to WARN in the REPL #6734

Closed
wants to merge 1 commit into from
Closed

[SPARK-7261][Core] Change default log level to WARN in the REPL #6734

wants to merge 1 commit into from

Conversation

zsxwing
Copy link
Member

@zsxwing zsxwing commented Jun 10, 2015

  1. Add log4j-defaults-repl.properties that has log level WARN.
  2. When logging is initialized, check whether inside the REPL. If so, use log4j-defaults-repl.properties.
  3. Print the following information if using log4j-defaults-repl.properties:
Using Spark's repl log4j profile: org/apache/spark/log4j-defaults-repl.properties
To adjust logging level use sc.setLogLevel("INFO")

@SparkQA
Copy link

SparkQA commented Jun 10, 2015

Test build #34552 has finished for PR 6734 at commit c42e643.

  • This patch fails RAT tests.
  • This patch merges cleanly.
  • This patch adds no public classes.

@SparkQA
Copy link

SparkQA commented Jun 10, 2015

Test build #34553 has finished for PR 6734 at commit 2cb5bbc.

  • This patch passes all tests.
  • This patch merges cleanly.
  • This patch adds no public classes.

@SparkQA
Copy link

SparkQA commented Jun 10, 2015

Test build #34558 has finished for PR 6734 at commit 3835eff.

  • This patch passes all tests.
  • This patch merges cleanly.
  • This patch adds no public classes.

@pwendell
Copy link
Contributor

This looks good, but will wait until #6735 is merged for final sign off.

interpClass.getMethod("interp").invoke(null) != null
} catch {
case e: ClassNotFoundException => false
}
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I'll update this myself

@andrewor14
Copy link
Contributor

LGTM merging into master

@asfgit asfgit closed this in 80043e9 Jun 10, 2015
@zsxwing zsxwing deleted the log4j-repl branch June 11, 2015 03:50
nemccarthy pushed a commit to nemccarthy/spark that referenced this pull request Jun 19, 2015
1. Add `log4j-defaults-repl.properties` that has log level WARN.
2. When logging is initialized, check whether inside the REPL. If so, use `log4j-defaults-repl.properties`.
3. Print the following information if using `log4j-defaults-repl.properties`:
```
Using Spark's repl log4j profile: org/apache/spark/log4j-defaults-repl.properties
To adjust logging level use sc.setLogLevel("INFO")
```

Author: zsxwing <zsxwing@gmail.com>

Closes apache#6734 from zsxwing/log4j-repl and squashes the following commits:

3835eff [zsxwing] Change default log level to WARN in the REPL
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
4 participants