Skip to content

SPARK-7669 Builds against Hadoop 2.6+ get inconsistent curator depend… - #6191

Closed
steveloughran wants to merge 2 commits into
apache:masterfrom
steveloughran:stevel/SPARK-7669-hadoop-2.6
Closed

steveloughran wants to merge 2 commits into
apache:masterfrom
steveloughran:stevel/SPARK-7669-hadoop-2.6

Conversation

@steveloughran

Copy link
Copy Markdown
Contributor

This adds a new profile, hadoop-2.6, copying over the hadoop-2.4 properties, updating ZK to 3.4.6 and making the curator version a configurable option. That keeps the curator-recipes JAR in sync with that used in hadoop.

There's one more option to consider: making the full curator-client version explicit with its own dependency version. This will pin down the version from hadoop and hive imports

@AmplabJenkins

Copy link
Copy Markdown

Merged build triggered.

@AmplabJenkins

Copy link
Copy Markdown

Merged build started.

@SparkQA

SparkQA commented May 15, 2015

Copy link
Copy Markdown

Test build #32833 has started for PR 6191 at commit 2901ea9.

Comment thread pom.xml

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

LGTM but do we need to explicitly manage curator-client versions too for consistency? just manage, not actually make a dependency on anywhere.

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

that's what I meantioned as an extended patch, I do think it's the right way to do it.

I've just pushed up a a new commit which declared the version of curator-framework and curator-client, so they will always be consistent and under the control of the spark build. It doesn't scope the new JARs, on the basis that this POM is only setting the versions. They could be given the hadoop.deps.scope though

@SparkQA

SparkQA commented May 15, 2015

Copy link
Copy Markdown

Test build #32833 has finished for PR 6191 at commit 2901ea9.

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

@AmplabJenkins

Copy link
Copy Markdown

Merged build finished. Test PASSed.

@AmplabJenkins

Copy link
Copy Markdown

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

@AmplabJenkins

Copy link
Copy Markdown

Merged build triggered.

@AmplabJenkins

Copy link
Copy Markdown

Merged build started.

@SparkQA

SparkQA commented May 15, 2015

Copy link
Copy Markdown

Test build #32847 has started for PR 6191 at commit e3e281a.

@SparkQA

SparkQA commented May 15, 2015

Copy link
Copy Markdown

Test build #32847 has finished for PR 6191 at commit e3e281a.

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

@AmplabJenkins

Copy link
Copy Markdown

Merged build finished. Test PASSed.

@AmplabJenkins

Copy link
Copy Markdown

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

@srowen

srowen commented May 16, 2015

Copy link
Copy Markdown
Member

LGTM, will leave it for comments a short while.

asfgit pushed a commit that referenced this pull request May 17, 2015
This adds a new profile, `hadoop-2.6`, copying over the hadoop-2.4 properties, updating ZK to 3.4.6 and making the curator version a configurable option. That keeps the curator-recipes JAR in sync with that used in hadoop.

There's one more option to consider: making the full curator-client version explicit with its own dependency version. This will pin down the version from hadoop and hive imports

Author: Steve Loughran <stevel@hortonworks.com>

Closes #6191 from steveloughran/stevel/SPARK-7669-hadoop-2.6 and squashes the following commits:

e3e281a [Steve Loughran] SPARK-7669 declare the version of curator-client and curator-framework JARs
2901ea9 [Steve Loughran] SPARK-7669 Builds against Hadoop 2.6+ get inconsistent curator dependencies

(cherry picked from commit 5021766)
Signed-off-by: Sean Owen <sowen@cloudera.com>
@asfgit asfgit closed this in 5021766 May 17, 2015
@steveloughran
steveloughran deleted the stevel/SPARK-7669-hadoop-2.6 branch May 19, 2015 09:06
jeanlyn pushed a commit to jeanlyn/spark that referenced this pull request May 28, 2015
This adds a new profile, `hadoop-2.6`, copying over the hadoop-2.4 properties, updating ZK to 3.4.6 and making the curator version a configurable option. That keeps the curator-recipes JAR in sync with that used in hadoop.

There's one more option to consider: making the full curator-client version explicit with its own dependency version. This will pin down the version from hadoop and hive imports

Author: Steve Loughran <stevel@hortonworks.com>

Closes apache#6191 from steveloughran/stevel/SPARK-7669-hadoop-2.6 and squashes the following commits:

e3e281a [Steve Loughran] SPARK-7669 declare the version of curator-client and curator-framework JARs
2901ea9 [Steve Loughran] SPARK-7669 Builds against Hadoop 2.6+ get inconsistent curator dependencies
jeanlyn pushed a commit to jeanlyn/spark that referenced this pull request Jun 12, 2015
This adds a new profile, `hadoop-2.6`, copying over the hadoop-2.4 properties, updating ZK to 3.4.6 and making the curator version a configurable option. That keeps the curator-recipes JAR in sync with that used in hadoop.

There's one more option to consider: making the full curator-client version explicit with its own dependency version. This will pin down the version from hadoop and hive imports

Author: Steve Loughran <stevel@hortonworks.com>

Closes apache#6191 from steveloughran/stevel/SPARK-7669-hadoop-2.6 and squashes the following commits:

e3e281a [Steve Loughran] SPARK-7669 declare the version of curator-client and curator-framework JARs
2901ea9 [Steve Loughran] SPARK-7669 Builds against Hadoop 2.6+ get inconsistent curator dependencies
nemccarthy pushed a commit to nemccarthy/spark that referenced this pull request Jun 19, 2015
This adds a new profile, `hadoop-2.6`, copying over the hadoop-2.4 properties, updating ZK to 3.4.6 and making the curator version a configurable option. That keeps the curator-recipes JAR in sync with that used in hadoop.

There's one more option to consider: making the full curator-client version explicit with its own dependency version. This will pin down the version from hadoop and hive imports

Author: Steve Loughran <stevel@hortonworks.com>

Closes apache#6191 from steveloughran/stevel/SPARK-7669-hadoop-2.6 and squashes the following commits:

e3e281a [Steve Loughran] SPARK-7669 declare the version of curator-client and curator-framework JARs
2901ea9 [Steve Loughran] SPARK-7669 Builds against Hadoop 2.6+ get inconsistent curator dependencies
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