Skip to content

Conversation

@srowen
Copy link
Member

@srowen srowen commented Nov 8, 2014

SPARK-3797 introduced the network/yarn module, but its YARN code depends on YARN APIs not present in older versions covered by the yarn-alpha profile. As a result builds like mvn -Pyarn-alpha -Phadoop-0.23 -Dhadoop.version=0.23.7 -DskipTests clean package fail.

The solution is just to not build network/yarn with profile yarn-alpha.

@SparkQA
Copy link

SparkQA commented Nov 8, 2014

Test build #23094 has started for PR 3167 at commit 88938cb.

  • This patch merges cleanly.

@SparkQA
Copy link

SparkQA commented Nov 8, 2014

Test build #23094 has finished for PR 3167 at commit 88938cb.

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

@tgravescs
Copy link
Contributor

+1. looks good.

asfgit pushed a commit that referenced this pull request Nov 11, 2014
… module

SPARK-3797 introduced the `network/yarn` module, but its YARN code depends on YARN APIs not present in older versions covered by the `yarn-alpha` profile. As a result builds like `mvn -Pyarn-alpha -Phadoop-0.23 -Dhadoop.version=0.23.7 -DskipTests clean package` fail.

The solution is just to not build `network/yarn` with profile `yarn-alpha`.

Author: Sean Owen <sowen@cloudera.com>

Closes #3167 from srowen/SPARK-4305 and squashes the following commits:

88938cb [Sean Owen] Don't build network/yarn in yarn-alpha profile as it won't compile

(cherry picked from commit f820b56)
Signed-off-by: Thomas Graves <tgraves@apache.org>
@asfgit asfgit closed this in f820b56 Nov 11, 2014
@srowen srowen deleted the SPARK-4305 branch November 19, 2014 08:26
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