aboutsummaryrefslogtreecommitdiff
path: root/core/src/test
diff options
context:
space:
mode:
authorXin Wu <xinwu@us.ibm.com>2016-06-03 14:26:48 -0700
committerMarcelo Vanzin <vanzin@cloudera.com>2016-06-03 14:26:48 -0700
commit28ad0f7b0dc7bf24fac251c4f131aca74ba1c1d2 (patch)
tree5c5053c797f3c7d4184a53949970cf2229a97cb6 /core/src/test
parent61b80d552aafb262b5f817f7bc9c0acd0328715b (diff)
downloadspark-28ad0f7b0dc7bf24fac251c4f131aca74ba1c1d2.tar.gz
spark-28ad0f7b0dc7bf24fac251c4f131aca74ba1c1d2.tar.bz2
spark-28ad0f7b0dc7bf24fac251c4f131aca74ba1c1d2.zip
[SPARK-15681][CORE] allow lowercase or mixed case log level string when calling sc.setLogLevel
## What changes were proposed in this pull request? Currently `SparkContext API setLogLevel(level: String) `can not handle lower case or mixed case input string. But `org.apache.log4j.Level.toLevel` can take lowercase or mixed case. This PR is to allow case-insensitive user input for the log level. ## How was this patch tested? A unit testcase is added. Author: Xin Wu <xinwu@us.ibm.com> Closes #13422 from xwu0226/reset_loglevel.
Diffstat (limited to 'core/src/test')
-rw-r--r--core/src/test/scala/org/apache/spark/SparkContextSuite.scala15
1 files changed, 15 insertions, 0 deletions
diff --git a/core/src/test/scala/org/apache/spark/SparkContextSuite.scala b/core/src/test/scala/org/apache/spark/SparkContextSuite.scala
index ae665138b9..4fa3cab181 100644
--- a/core/src/test/scala/org/apache/spark/SparkContextSuite.scala
+++ b/core/src/test/scala/org/apache/spark/SparkContextSuite.scala
@@ -363,4 +363,19 @@ class SparkContextSuite extends SparkFunSuite with LocalSparkContext {
sc.stop()
assert(result == null)
}
+
+ test("log level case-insensitive and reset log level") {
+ sc = new SparkContext(new SparkConf().setAppName("test").setMaster("local"))
+ val originalLevel = org.apache.log4j.Logger.getRootLogger().getLevel
+ try {
+ sc.setLogLevel("debug")
+ assert(org.apache.log4j.Logger.getRootLogger().getLevel === org.apache.log4j.Level.DEBUG)
+ sc.setLogLevel("INfo")
+ assert(org.apache.log4j.Logger.getRootLogger().getLevel === org.apache.log4j.Level.INFO)
+ } finally {
+ sc.setLogLevel(originalLevel.toString)
+ assert(org.apache.log4j.Logger.getRootLogger().getLevel === originalLevel)
+ sc.stop()
+ }
+ }
}