Skip to content

Conversation

@liancheng
Copy link
Contributor

When refactoring SQL options from plain strings to the strongly typed SQLConfEntry, spark.sql.hive.version wasn't migrated, and doesn't show up in the result of SET -v, as SET -v only shows public SQLConfEntry instances. This affects compatibility with Simba ODBC driver.

This PR migrates this SQL option as a SQLConfEntry to fix this issue.

@yhuai
Copy link
Contributor

yhuai commented Sep 27, 2015

LGTM

@liancheng
Copy link
Contributor Author

Thanks for the review. Merging to master.

@asfgit asfgit closed this in 6f94d56 Sep 27, 2015
@SparkQA
Copy link

SparkQA commented Sep 27, 2015

Test build #43052 has finished for PR 8925 at commit f90be1a.

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

@liancheng liancheng deleted the spark-10845/hive-version-conf branch September 28, 2015 17:35
asfgit pushed a commit that referenced this pull request Oct 14, 2015
When refactoring SQL options from plain strings to the strongly typed `SQLConfEntry`, `spark.sql.hive.version` wasn't migrated, and doesn't show up in the result of `SET -v`, as `SET -v` only shows public `SQLConfEntry` instances. This affects compatibility with Simba ODBC driver.

This PR migrates this SQL option as a `SQLConfEntry` to fix this issue.

Author: Cheng Lian <[email protected]>

Closes #8925 from liancheng/spark-10845/hive-version-conf.

(cherry picked from commit 6f94d56)
Signed-off-by: Reynold Xin <[email protected]>
@rxin
Copy link
Contributor

rxin commented Oct 14, 2015

I backported this into branch-1.5.

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.

4 participants