Skip to content

[SPARK-14994][SQL] Remove execution hive from HiveSessionState #12770

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.

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

Closed
wants to merge 14 commits into from

Conversation

rxin
Copy link
Contributor

@rxin rxin commented Apr 28, 2016

What changes were proposed in this pull request?

This patch removes executionHive from HiveSessionState and HiveSharedState.

How was this patch tested?

Updated test cases.

@rxin
Copy link
Contributor Author

rxin commented Apr 28, 2016

Note that this builds on #12722.

@SparkQA
Copy link

SparkQA commented Apr 29, 2016

Test build #57287 has finished for PR 12770 at commit 045865d.

  • This patch fails Spark unit tests.
  • This patch merges cleanly.
  • This patch adds no public classes.

@SparkQA
Copy link

SparkQA commented Apr 29, 2016

Test build #57309 has finished for PR 12770 at commit 2a84533.

  • This patch fails Spark unit tests.
  • This patch merges cleanly.
  • This patch adds no public classes.

@SparkQA
Copy link

SparkQA commented Apr 29, 2016

Test build #57313 has finished for PR 12770 at commit b74f5a5.

  • This patch fails MiMa tests.
  • This patch merges cleanly.
  • This patch adds no public classes.

@SparkQA
Copy link

SparkQA commented Apr 29, 2016

Test build #2931 has finished for PR 12770 at commit b74f5a5.

  • This patch fails Spark unit tests.
  • This patch merges cleanly.
  • This patch adds no public classes.

@SparkQA
Copy link

SparkQA commented Apr 29, 2016

Test build #57316 has finished for PR 12770 at commit e68b453.

  • This patch fails Spark unit tests.
  • This patch merges cleanly.
  • This patch adds no public classes.

@SparkQA
Copy link

SparkQA commented Apr 29, 2016

Test build #2932 has finished for PR 12770 at commit e68b453.

  • This patch fails Spark unit tests.
  • This patch merges cleanly.
  • This patch adds no public classes.

@SparkQA
Copy link

SparkQA commented Apr 29, 2016

Test build #2934 has finished for PR 12770 at commit e68b453.

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

@SparkQA
Copy link

SparkQA commented Apr 29, 2016

Test build #2935 has finished for PR 12770 at commit e68b453.

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

@SparkQA
Copy link

SparkQA commented Apr 29, 2016

Test build #2933 has finished for PR 12770 at commit e68b453.

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

@SparkQA
Copy link

SparkQA commented Apr 29, 2016

Test build #57318 has finished for PR 12770 at commit bf03065.

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

@rxin
Copy link
Contributor Author

rxin commented Apr 29, 2016

Merging in master.

@asfgit asfgit closed this in 054f991 Apr 29, 2016
@SparkQA
Copy link

SparkQA commented Apr 29, 2016

Test build #2939 has finished for PR 12770 at commit bf03065.

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

val WAREHOUSE_PATH = SQLConfigBuilder("spark.sql.warehouse.dir")
.doc("The default location for managed databases and tables.")
.stringConf
.createWithDefault("${system:user.dir}/spark-warehouse")
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@rxin I think default for this should be something else. This will fail things like https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/58168/

asfgit pushed a commit that referenced this pull request Jan 7, 2017
## What changes were proposed in this pull request?

This fix is related to be bug: https://issues.apache.org/jira/browse/SPARK-18372 .
The insertIntoHiveTable would generate a .staging directory, but this directory  fail to be removed in the end.

This is backport from spark 2.0.x code, and is related to PR #12770

## How was this patch tested?
manual tests

Author: Mingjie Tang <mtanghortonworks.com>

Author: Mingjie Tang <mtang@hortonworks.com>
Author: Mingjie Tang <mtang@HW12398.local>

Closes #15819 from merlintang/branch-1.6.
zzcclp pushed a commit to zzcclp/spark that referenced this pull request Jan 7, 2017
## What changes were proposed in this pull request?

This fix is related to be bug: https://issues.apache.org/jira/browse/SPARK-18372 .
The insertIntoHiveTable would generate a .staging directory, but this directory  fail to be removed in the end.

This is backport from spark 2.0.x code, and is related to PR apache#12770

## How was this patch tested?
manual tests

Author: Mingjie Tang <mtanghortonworks.com>

Author: Mingjie Tang <mtang@hortonworks.com>
Author: Mingjie Tang <mtang@HW12398.local>

Closes apache#15819 from merlintang/branch-1.6.

(cherry picked from commit 2303887)
dosoft pushed a commit to WANdisco/spark that referenced this pull request Jan 24, 2017
## What changes were proposed in this pull request?

This fix is related to be bug: https://issues.apache.org/jira/browse/SPARK-18372 .
The insertIntoHiveTable would generate a .staging directory, but this directory  fail to be removed in the end.

This is backport from spark 2.0.x code, and is related to PR apache#12770

## How was this patch tested?
manual tests

Author: Mingjie Tang <mtanghortonworks.com>

Author: Mingjie Tang <mtang@hortonworks.com>
Author: Mingjie Tang <mtang@HW12398.local>

Closes apache#15819 from merlintang/branch-1.6.
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