Skip to content

Conversation

@wangyum
Copy link
Member

@wangyum wangyum commented Mar 5, 2018

What changes were proposed in this pull request?

Update spark.sql.hive.metastore.version to 2.3.2, same as HiveUtils.scala:

val HIVE_METASTORE_VERSION = buildConf("spark.sql.hive.metastore.version")
.doc("Version of the Hive metastore. Available options are " +
s"<code>0.12.0</code> through <code>2.3.2</code>.")

How was this patch tested?

N/A

@hvanhovell
Copy link
Contributor

LGTM :)

@SparkQA
Copy link

SparkQA commented Mar 5, 2018

Test build #87955 has finished for PR 20734 at commit 1052c72.

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

@hvanhovell
Copy link
Contributor

Merging to master. Thanks!

@asfgit asfgit closed this in 947b4e6 Mar 5, 2018
@wangyum wangyum deleted the SPARK-23510-FOLLOW-UP branch March 5, 2018 15:43
mgaido91 pushed a commit to mgaido91/spark that referenced this pull request Mar 5, 2018
## What changes were proposed in this pull request?
Update `spark.sql.hive.metastore.version` to 2.3.2, same as HiveUtils.scala:
https://github.com/apache/spark/blob/ff1480189b827af0be38605d566a4ee71b4c36f6/sql/hive/src/main/scala/org/apache/spark/sql/hive/HiveUtils.scala#L63-L65

## How was this patch tested?

N/A

Author: Yuming Wang <[email protected]>

Closes apache#20734 from wangyum/SPARK-23510-FOLLOW-UP.
tkakantousis pushed a commit to tkakantousis/spark that referenced this pull request Mar 11, 2018
## What changes were proposed in this pull request?
Update `spark.sql.hive.metastore.version` to 2.3.2, same as HiveUtils.scala:
https://github.com/apache/spark/blob/ff1480189b827af0be38605d566a4ee71b4c36f6/sql/hive/src/main/scala/org/apache/spark/sql/hive/HiveUtils.scala#L63-L65

## How was this patch tested?

N/A

Author: Yuming Wang <[email protected]>

Closes apache#20734 from wangyum/SPARK-23510-FOLLOW-UP.
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 this pull request may close these issues.

3 participants