Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

[SQL] Use HiveContext's sessionState in HiveMetastoreCatalog.hiveDefaultTableFilePath #4355

Closed
wants to merge 1 commit into from
Closed

[SQL] Use HiveContext's sessionState in HiveMetastoreCatalog.hiveDefaultTableFilePath #4355

wants to merge 1 commit into from

Conversation

yhuai
Copy link
Contributor

@yhuai yhuai commented Feb 4, 2015

client.getDatabaseCurrent uses SessionState's local variable which can be an issue.

@SparkQA
Copy link

SparkQA commented Feb 4, 2015

Test build #26724 has started for PR 4355 at commit 84a29e5.

  • This patch merges cleanly.

@SparkQA
Copy link

SparkQA commented Feb 4, 2015

Test build #26724 has finished for PR 4355 at commit 84a29e5.

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

@AmplabJenkins
Copy link

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/26724/
Test PASSed.

asfgit pushed a commit that referenced this pull request Feb 4, 2015
@yhuai @marmbrus
[SQL] Use HiveContext's sessionState in HiveMetastoreCatalog.hiveDefa…
cb4c3e5 
…ultTableFilePath

`client.getDatabaseCurrent` uses SessionState's local variable which can be an issue.

Author: Yin Huai <yhuai@databricks.com>

Closes #4355 from yhuai/defaultTablePath and squashes the following commits:

84a29e5 [Yin Huai] Use HiveContext's sessionState instead of using SessionState's thread local variable.

(cherry picked from commit 548c9c2)
Signed-off-by: Michael Armbrust <michael@databricks.com>
@marmbrus
Copy link
Contributor

marmbrus commented Feb 4, 2015

Thanks! Merged to master.

@asfgit asfgit closed this in 548c9c2 Feb 4, 2015
@yhuai yhuai deleted the defaultTablePath branch February 11, 2015 18:21
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
4 participants
@yhuai @SparkQA @AmplabJenkins @marmbrus