[SPARK-19664] put 'hive.metastore.warehouse.dir' in hadoopConf place

• Type: Improvement
• Status: Resolved
• Priority: Minor
• Resolution: Fixed
• Affects Version/s: 2.1.0
• Fix Version/s: 2.2.0
• Component/s: SQL
• Assignee: Song Jun
• Reporter: Song Jun
• Created: 20/Feb/17 04:57
• Updated: 24/Feb/17 06:58
• Resolved: 24/Feb/17 06:57

Description

In SPARK-15959, we bring back the 'hive.metastore.warehouse.dir' , while in the logic, when use the value of 'spark.sql.warehouse.dir' to overwrite 'spark.sql.warehouse.dir' , it set it to 'sparkContext.conf' , I think it should put in 'sparkContext.hadoopConfiguration' and overwrite the original value of hadoopConf

https://github.com/apache/spark/blob/master/sql/core/src/main/scala/org/apache/spark/sql/internal/SharedState.scala#L64

Issue Links

• links to: [Github] Pull Request #16996 (windpiger)