Skip to content

Conversation

@liancheng
Copy link
Contributor

This PR disables HiveThriftServer2 asynchronous execution by setting runInBackground argument in ExecuteStatementOperation to false, and reverting SparkExecuteStatementOperation.run in Hive 13 shim to Hive 12 version. This change makes Simba ODBC driver v1.0.0.1000 work.

Review on Reviewable

@SparkQA
Copy link

SparkQA commented Nov 28, 2014

Test build #23936 has started for PR 3506 at commit 593804d.

  • This patch merges cleanly.

@liancheng liancheng changed the title Disables asynchronous execution in Hive 0.13.1 HiveThriftServer2 [SPARK-4645][SQL] Disables asynchronous execution in Hive 0.13.1 HiveThriftServer2 Nov 28, 2014
@SparkQA
Copy link

SparkQA commented Nov 28, 2014

Test build #23936 has finished for PR 3506 at commit 593804d.

  • 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/23936/
Test PASSed.

@pwendell
Copy link
Contributor

Thanks Cheng - I'll pull this in.

@asfgit asfgit closed this in 5b99bf2 Nov 28, 2014
asfgit pushed a commit that referenced this pull request Nov 28, 2014
…ThriftServer2

This PR disables HiveThriftServer2 asynchronous execution by setting `runInBackground` argument in `ExecuteStatementOperation` to `false`, and reverting `SparkExecuteStatementOperation.run` in Hive 13 shim to Hive 12 version. This change makes Simba ODBC driver v1.0.0.1000 work.

<!-- Reviewable:start -->
[<img src="https://reviewable.io/review_button.png" height=40 alt="Review on Reviewable"/>](https://reviewable.io/reviews/apache/spark/3506)
<!-- Reviewable:end -->

Author: Cheng Lian <[email protected]>

Closes #3506 from liancheng/disable-async-exec and squashes the following commits:

593804d [Cheng Lian] Disables asynchronous execution in Hive 0.13.1 HiveThriftServer2
@liancheng liancheng deleted the disable-async-exec branch November 29, 2014 04:19
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