Skip to content

Conversation

@vanzin
Copy link
Contributor

@vanzin vanzin commented May 14, 2014

No description provided.

@vanzin
Copy link
Contributor Author

vanzin commented May 14, 2014

@srowen this fixes something that was still broken for me after c7253da

@AmplabJenkins
Copy link

Can one of the admins verify this patch?

@srowen
Copy link
Member

srowen commented May 14, 2014

I see, so it's definitely avro-ipc that pulls it in? Makes sense. If you're not certain, leave them both. But this is good if it's only avro-ipc.

@pwendell
Copy link
Contributor

I dug around through several different avro versions and it does seem that consistently avro-ipc brings in netty and avro doesn't directly bring it in.

@pwendell
Copy link
Contributor

I'll pull this in, thanks!

@asfgit asfgit closed this in 54ae832 May 14, 2014
asfgit pushed a commit that referenced this pull request May 14, 2014
Author: Marcelo Vanzin <[email protected]>

Closes #763 from vanzin/netty-dep-hell and squashes the following commits:

dfb6ce2 [Marcelo Vanzin] Fix dep exclusion: avro-ipc, not avro, depends on netty.
@vanzin vanzin deleted the netty-dep-hell branch June 9, 2014 18:58
pdeyhim pushed a commit to pdeyhim/spark-1 that referenced this pull request Jun 25, 2014
Author: Marcelo Vanzin <[email protected]>

Closes apache#763 from vanzin/netty-dep-hell and squashes the following commits:

dfb6ce2 [Marcelo Vanzin] Fix dep exclusion: avro-ipc, not avro, depends on netty.
turboFei pushed a commit to turboFei/spark that referenced this pull request Nov 6, 2025
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