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

Update hadoop to 2.7.2 #262

Merged
merged 4 commits into from
Mar 18, 2016
Merged

Update hadoop to 2.7.2 #262

merged 4 commits into from
Mar 18, 2016

Conversation

ceefour
Copy link
Contributor

@ceefour ceefour commented Mar 15, 2016

No description provided.

@ceefour
Copy link
Contributor Author

ceefour commented Mar 15, 2016

It's weird that hadoop-client and hadoop-hdfs do not agree on netty version. This should be fixable.

@ceefour
Copy link
Contributor Author

ceefour commented Mar 15, 2016

Root issue reported to Hadoop project: apache/hadoop#84 so in the future we can upgrade easily

@steveloughran
Copy link

Why the two versions? hdfs netty went in later, MR netty hasn't (yet) updated, then there's the fear of breaking things downstream. Hadoop 2.8 splits hdfs client libs out into their own JAR; there's no dependency on netty 4 in hadoop-hdfs-client at all.

@spmallette
Copy link
Contributor

@okram i think you have to lead the review on this hadoop stuff.

@okram
Copy link
Contributor

okram commented Mar 18, 2016

VOTE +1.

I merged locally (had to tweak a few other things -- docs and Manifest stuff, but works). This appears okay for tp31 and an upmerge to master.

@spmallette
Copy link
Contributor

VOTE +1

@dkuppitz
Copy link
Contributor

VOTE: +1

@asfgit asfgit merged commit 9154660 into apache:tp31 Mar 18, 2016
@okram
Copy link
Contributor

okram commented Mar 18, 2016

Note that I also verified that this still works with Hadoop 2.7.1 by building the docs with Hadoop 2.7.1 running.

@okram
Copy link
Contributor

okram commented Mar 18, 2016

Note that I also just merged master/ and all is good in the (proverbial) hood.

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.

6 participants