You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@spark.apache.org by gu...@apache.org on 2017/12/09 11:20:38 UTC
spark git commit: [SPARK-22672][SQL][TEST][FOLLOWUP] Fix to use
`spark.conf`
Repository: spark
Updated Branches:
refs/heads/master acf7ef315 -> 251b2c03b
[SPARK-22672][SQL][TEST][FOLLOWUP] Fix to use `spark.conf`
## What changes were proposed in this pull request?
During https://github.com/apache/spark/pull/19882, `conf` is mistakenly used to switch ORC implementation between `native` and `hive`. To affect `OrcTest` correctly, `spark.conf` should be used.
## How was this patch tested?
Pass the tests.
Author: Dongjoon Hyun <do...@apache.org>
Closes #19931 from dongjoon-hyun/SPARK-22672-2.
Project: http://git-wip-us.apache.org/repos/asf/spark/repo
Commit: http://git-wip-us.apache.org/repos/asf/spark/commit/251b2c03
Tree: http://git-wip-us.apache.org/repos/asf/spark/tree/251b2c03
Diff: http://git-wip-us.apache.org/repos/asf/spark/diff/251b2c03
Branch: refs/heads/master
Commit: 251b2c03b4b2846c9577390f87db3db511be7721
Parents: acf7ef3
Author: Dongjoon Hyun <do...@apache.org>
Authored: Sat Dec 9 20:20:28 2017 +0900
Committer: hyukjinkwon <gu...@gmail.com>
Committed: Sat Dec 9 20:20:28 2017 +0900
----------------------------------------------------------------------
.../apache/spark/sql/execution/datasources/orc/OrcTest.scala | 6 +++---
1 file changed, 3 insertions(+), 3 deletions(-)
----------------------------------------------------------------------
http://git-wip-us.apache.org/repos/asf/spark/blob/251b2c03/sql/core/src/test/scala/org/apache/spark/sql/execution/datasources/orc/OrcTest.scala
----------------------------------------------------------------------
diff --git a/sql/core/src/test/scala/org/apache/spark/sql/execution/datasources/orc/OrcTest.scala b/sql/core/src/test/scala/org/apache/spark/sql/execution/datasources/orc/OrcTest.scala
index d94cb85..38b34a0 100644
--- a/sql/core/src/test/scala/org/apache/spark/sql/execution/datasources/orc/OrcTest.scala
+++ b/sql/core/src/test/scala/org/apache/spark/sql/execution/datasources/orc/OrcTest.scala
@@ -51,12 +51,12 @@ abstract class OrcTest extends QueryTest with SQLTestUtils with BeforeAndAfterAl
protected override def beforeAll(): Unit = {
super.beforeAll()
- originalConfORCImplementation = conf.getConf(ORC_IMPLEMENTATION)
- conf.setConf(ORC_IMPLEMENTATION, orcImp)
+ originalConfORCImplementation = spark.conf.get(ORC_IMPLEMENTATION)
+ spark.conf.set(ORC_IMPLEMENTATION.key, orcImp)
}
protected override def afterAll(): Unit = {
- conf.setConf(ORC_IMPLEMENTATION, originalConfORCImplementation)
+ spark.conf.set(ORC_IMPLEMENTATION.key, originalConfORCImplementation)
super.afterAll()
}
---------------------------------------------------------------------
To unsubscribe, e-mail: commits-unsubscribe@spark.apache.org
For additional commands, e-mail: commits-help@spark.apache.org