Skip to content
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

[SPARK-4177][Doc]update build doc since JDBC/CLI support hive 13 now #3042

Closed
wants to merge 2 commits into from

Conversation

scwf
Copy link
Contributor

@scwf scwf commented Oct 31, 2014

No description provided.

@scwf scwf changed the title [Doc]update build doc since JDBC/CLI support hive 13 [Doc]update build doc since JDBC/CLI support hive 13 now Oct 31, 2014
@SparkQA
Copy link

SparkQA commented Oct 31, 2014

Test build #22640 has started for PR 3042 at commit 1891553.

  • This patch merges cleanly.

@SparkQA
Copy link

SparkQA commented Oct 31, 2014

Test build #22640 has finished for PR 3042 at commit 1891553.

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

@AmplabJenkins
Copy link

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

@scwf
Copy link
Contributor Author

scwf commented Oct 31, 2014

Failed due to #3041
retest this please.

@scwf scwf changed the title [Doc]update build doc since JDBC/CLI support hive 13 now [SPARK-4177][Doc]update build doc since JDBC/CLI support hive 13 now Oct 31, 2014
@scwf
Copy link
Contributor Author

scwf commented Nov 1, 2014

@pwendell, is this ok to go?

@@ -105,8 +105,7 @@ mvn -Pyarn-alpha -Phadoop-2.3 -Dhadoop.version=2.3.0 -Dyarn.version=0.23.7 -Dski
To enable Hive integration for Spark SQL along with its JDBC server and CLI,
add the `-Phive` profile to your existing build options. By default Spark
will build with Hive 0.13.1 bindings. You can also build for Hive 0.12.0 using
the `-Phive-0.12.0` profile. NOTE: currently the JDBC server is only
supported for Hive 0.12.0.
the `-Phive-0.12.0` profile.
Copy link
Contributor

Choose a reason for hiding this comment

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

can you remove the TODO above?

@pwendell
Copy link
Contributor

pwendell commented Nov 2, 2014

Looks good pending a minor comment.

@SparkQA
Copy link

SparkQA commented Nov 2, 2014

Test build #22719 has started for PR 3042 at commit 3784ed1.

  • This patch merges cleanly.

@SparkQA
Copy link

SparkQA commented Nov 2, 2014

Test build #22719 has finished for PR 3042 at commit 3784ed1.

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

@scwf
Copy link
Contributor Author

scwf commented Nov 3, 2014

@pwendell, this should be ready to go.

@pwendell
Copy link
Contributor

pwendell commented Nov 3, 2014

Thanks - I pulled this in!

@asfgit asfgit closed this in 001acc4 Nov 3, 2014
@scwf scwf deleted the patch-9 branch November 4, 2014 01:49
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