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.

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

[DOC][Build] Wrong cmd for build spark with apache hadoop 2.4.X and hive 12 #3335

Closed
wants to merge 2 commits into from
Closed

[DOC][Build] Wrong cmd for build spark with apache hadoop 2.4.X and hive 12 #3335

wants to merge 2 commits into from

Conversation

scwf
Copy link
Contributor

@scwf scwf commented Nov 18, 2014

No description provided.

@SparkQA
Copy link

SparkQA commented Nov 18, 2014

Test build #23534 has started for PR 3335 at commit 60d595e.

  • This patch merges cleanly.

@SparkQA
Copy link

SparkQA commented Nov 18, 2014

Test build #23534 has finished for PR 3335 at commit 60d595e.

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

@scwf scwf changed the title [DOC] Wrong cmd for build spark with apache hadoop 2.4.X and hive 12 [DOC][Build] Wrong cmd for build spark with apache hadoop 2.4.X and hive 12 Nov 18, 2014
@scwf
Copy link
Contributor Author

scwf commented Nov 19, 2014

@pwendell, this should go into master and branch-1.2

pwendell
pwendell reviewed Nov 19, 2014
View reviewed changes
docs/building-spark.md
@@ -109,7 +109,7 @@ Hive 0.12.0 using the `-Phive-0.12.0` profile.
mvn -Pyarn -Phadoop-2.4 -Dhadoop.version=2.4.0 -Phive -Phive-thriftserver -DskipTests clean package

# Apache Hadoop 2.4.X with Hive 12 support
mvn -Pyarn -Phive -Phive-thriftserver-0.12.0 -Phadoop-2.4 -Dhadoop.version=2.4.0 -Phive -Phive-thriftserver -DskipTests clean package
mvn -Pyarn -Phadoop-2.4 -Dhadoop.version=2.4.0 -Phive-0.12.0 -Phive-thriftserver -DskipTests clean package
Copy link
Contributor

Choose a reason for hiding this comment

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

Doesn't this also need -Phive?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Yes, my mistake, updated.

@SparkQA
Copy link

SparkQA commented Nov 19, 2014

Test build #23638 has started for PR 3335 at commit d343113.

  • This patch merges cleanly.

@SparkQA
Copy link

SparkQA commented Nov 20, 2014

Test build #23638 has finished for PR 3335 at commit d343113.

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

@scwf
Copy link
Contributor Author

scwf commented Nov 22, 2014

Ping

1 similar comment
@scwf
Copy link
Contributor Author

scwf commented Nov 25, 2014

Ping

@scwf
Copy link
Contributor Author

scwf commented Nov 25, 2014

Hi @pwendell, is this ok to go?

@pwendell
Copy link
Contributor

Pulling this in - thanks!

asfgit pushed a commit that referenced this pull request Nov 25, 2014
@scwf @pwendell
[DOC][Build] Wrong cmd for build spark with apache hadoop 2.4.X and h…
1f4d1ac 
…ive 12

Author: wangfei <[email protected]>

Closes #3335 from scwf/patch-10 and squashes the following commits:

d343113 [wangfei] add '-Phive'
60d595e [wangfei] [DOC] Wrong cmd for build spark with apache hadoop 2.4.X and Hive 12 support

(cherry picked from commit 0fe54cf)
Signed-off-by: Patrick Wendell <[email protected]>
@asfgit asfgit closed this in 0fe54cf Nov 25, 2014
@scwf scwf deleted the patch-10 branch November 25, 2014 06:43
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.
4 participants
@scwf @SparkQA @AmplabJenkins @pwendell