-
Notifications
You must be signed in to change notification settings - Fork 28.6k
[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
Conversation
Note that this builds on #12722. |
Test build #57287 has finished for PR 12770 at commit
|
Test build #57309 has finished for PR 12770 at commit
|
Test build #57313 has finished for PR 12770 at commit
|
Test build #2931 has finished for PR 12770 at commit
|
Test build #57316 has finished for PR 12770 at commit
|
Test build #2932 has finished for PR 12770 at commit
|
Test build #2934 has finished for PR 12770 at commit
|
Test build #2935 has finished for PR 12770 at commit
|
Test build #2933 has finished for PR 12770 at commit
|
Test build #57318 has finished for PR 12770 at commit
|
Merging in master. |
Test build #2939 has finished for PR 12770 at commit
|
val WAREHOUSE_PATH = SQLConfigBuilder("spark.sql.warehouse.dir") | ||
.doc("The default location for managed databases and tables.") | ||
.stringConf | ||
.createWithDefault("${system:user.dir}/spark-warehouse") |
There was a problem hiding this comment.
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/
## 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.
## 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)
## 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.
What changes were proposed in this pull request?
This patch removes executionHive from HiveSessionState and HiveSharedState.
How was this patch tested?
Updated test cases.