Skip to content

[SPARK-6317][SQL]Fixed HIVE console startup issue #5011

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 2 commits into from
Closed

[SPARK-6317][SQL]Fixed HIVE console startup issue #5011

wants to merge 2 commits into from

Conversation

vinodkc
Copy link
Contributor

@vinodkc vinodkc commented Mar 13, 2015

No description provided.

@AmplabJenkins
Copy link

Can one of the admins verify this patch?

@srowen
Copy link
Member

srowen commented Mar 13, 2015

ok to test

@SparkQA
Copy link

SparkQA commented Mar 13, 2015

Test build #28562 has started for PR 5011 at commit b4f5453.

  • This patch merges cleanly.

@srowen
Copy link
Member

srowen commented Mar 13, 2015

LGTM, except the new import should sort one line below now. If you get to it, great, otherwise I can fix on merge.

@vinodkc
Copy link
Contributor Author

vinodkc commented Mar 13, 2015

Changed the order.Please check
Thanks
On Mar 13, 2015 6:45 PM, "Sean Owen" notifications@github.com wrote:

LGTM, except the new import should sort one line below now. If you get to
it, great, otherwise I can fix on merge.


Reply to this email directly or view it on GitHub
#5011 (comment).

@SparkQA
Copy link

SparkQA commented Mar 13, 2015

Test build #28566 has started for PR 5011 at commit b43925f.

  • This patch merges cleanly.

@SparkQA
Copy link

SparkQA commented Mar 13, 2015

Test build #28562 has finished for PR 5011 at commit b4f5453.

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

@vinodkc
Copy link
Contributor Author

vinodkc commented Mar 13, 2015

If DSL support is already removed from 1.3 release, I shall modify the file and submit it.

@SparkQA
Copy link

SparkQA commented Mar 13, 2015

Test build #28566 has finished for PR 5011 at commit b43925f.

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

@liancheng
Copy link
Contributor

LGTM, merging to master. Thanks!

@asfgit asfgit closed this in e360d5e Mar 13, 2015
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.

5 participants