Skip to content

[SQL][Hiveconsole] Bring hive console code up to date and update README.md #4330

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
Closed
OopsOutOfMemory wants to merge 3 commits into from
Closed

[SQL][Hiveconsole] Bring hive console code up to date and update README.md #4330

OopsOutOfMemory wants to merge 3 commits into from
+11 −35

Conversation

@OopsOutOfMemory
Copy link
Contributor

@OopsOutOfMemory OopsOutOfMemory commented Feb 3, 2015

Add import org.apache.spark.sql.Dsl._ to make DSL query works.
Since queryExecution is not avaliable in DataFrame, so remove it.

@OopsOutOfMemory
Copy link
Contributor Author

OopsOutOfMemory commented Feb 3, 2015

/cc @rxin

@AmplabJenkins
Copy link

AmplabJenkins commented Feb 3, 2015

Can one of the admins verify this patch?

@rxin
Copy link
Contributor

rxin commented Feb 3, 2015

Jenkins, test this please.

rxin
rxin reviewed Feb 3, 2015
View reviewed changes
project/SparkBuild.scala Outdated
Copy link
Contributor

@rxin rxin Feb 3, 2015

Choose a reason for hiding this comment

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

@marmbrus you probably want to take a look at this ...

@SparkQA
Copy link

SparkQA commented Feb 3, 2015

Test build #26631 has started for PR 4330 at commit d23ee9f.

  • This patch merges cleanly.

@SparkQA
Copy link

SparkQA commented Feb 3, 2015

Test build #26631 has finished for PR 4330 at commit d23ee9f.

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

@AmplabJenkins
Copy link

AmplabJenkins commented Feb 3, 2015

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

@OopsOutOfMemory
Update SparkBuild.scala
46eb790 
revert removed import
@OopsOutOfMemory
Copy link
Contributor Author

OopsOutOfMemory commented Feb 3, 2015

retest this please

@marmbrus
Copy link
Contributor

marmbrus commented Feb 3, 2015

ok to test

@SparkQA
Copy link

SparkQA commented Feb 3, 2015

Test build #26669 has started for PR 4330 at commit 46eb790.

  • This patch merges cleanly.

@SparkQA
Copy link

SparkQA commented Feb 3, 2015

Test build #26669 has finished for PR 4330 at commit 46eb790.

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

@AmplabJenkins
Copy link

AmplabJenkins commented Feb 3, 2015

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

@marmbrus
Copy link
Contributor

marmbrus commented Feb 4, 2015

Thanks, merged to master and 1.3

asfgit pushed a commit that referenced this pull request Feb 4, 2015
@OopsOutOfMemory @marmbrus
[SQL][Hiveconsole] Bring hive console code up to date and update READ…
2cdcfe3 
…ME.md

Add `import org.apache.spark.sql.Dsl._` to make DSL query works.
Since queryExecution is not avaliable in DataFrame, so remove it.

Author: OopsOutOfMemory <victorshengli@126.com>
Author: Sheng, Li <OopsOutOfMemory@users.noreply.github.com>

Closes #4330 from OopsOutOfMemory/hiveconsole and squashes the following commits:

46eb790 [Sheng, Li] Update SparkBuild.scala
d23ee9f [OopsOutOfMemory] minor
d4dd593 [OopsOutOfMemory] refine hive console

(cherry picked from commit b73d5ff)
Signed-off-by: Michael Armbrust <michael@databricks.com>
@asfgit asfgit closed this in b73d5ff Feb 4, 2015
@OopsOutOfMemory OopsOutOfMemory deleted the hiveconsole branch February 5, 2015 00:36
@OopsOutOfMemory OopsOutOfMemory mentioned this pull request Feb 5, 2015
Closed
asfgit pushed a commit that referenced this pull request Feb 6, 2015
@OopsOutOfMemory @marmbrus
[SQL][HiveConsole][DOC] HiveConsole correct hiveconsole imports
2abaa6e 
Sorry for that PR #4330 has some mistakes.

I correct it....  so it works correctly now.

Author: OopsOutOfMemory <victorshengli@126.com>

Closes #4389 from OopsOutOfMemory/doc and squashes the following commits:

843eed9 [OopsOutOfMemory] correct hiveconsole imports

(cherry picked from commit b62c352)
Signed-off-by: Michael Armbrust <michael@databricks.com>
asfgit pushed a commit that referenced this pull request Feb 6, 2015
@OopsOutOfMemory @marmbrus
[SQL][HiveConsole][DOC] HiveConsole correct hiveconsole imports
b62c352 
Sorry for that PR #4330 has some mistakes.

I correct it....  so it works correctly now.

Author: OopsOutOfMemory <victorshengli@126.com>

Closes #4389 from OopsOutOfMemory/doc and squashes the following commits:

843eed9 [OopsOutOfMemory] correct hiveconsole imports
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

Development

Successfully merging this pull request may close these issues.

5 participants

@OopsOutOfMemory @AmplabJenkins @rxin @SparkQA @marmbrus