Netty 4.1.x

[Charles Allen <ch...@snap.com.INVALID>]

https://github.com/apache/incubator-druid/pull/6417 proposes upgrading to
netty 4.1.x

A lot of the prior issues are likely resolved. Things like java-util are
part of the druid repository now, the dependent libraries which were still
using 4.0.x are upgraded (in the PR) to ones using 4.1.x, and Spark's
laster major version (2.3.x) has netty 4.1

I propose giving netty 4.1.x another shot.

Sound good?

Charles Allen

Re: Netty 4.1.x

[Gian Merlino <gi...@apache.org>]

It sounds good to me.

BTW, we still use netty 3.x for http-client and so it's pretty pervasive.
It coexists with netty 4.x (the packages are different) so there isn't a
conflict at that level. But if we wanted to, like, _fully_ upgrade to netty
4.1.x then it'd involve porting over the http-client.

On Fri, Oct 5, 2018 at 11:20 AM Charles Allen
<ch...@snap.com.invalid> wrote:

> https://github.com/apache/incubator-druid/pull/6417 proposes upgrading to
> netty 4.1.x
>
> A lot of the prior issues are likely resolved. Things like java-util are
> part of the druid repository now, the dependent libraries which were still
> using 4.0.x are upgraded (in the PR) to ones using 4.1.x, and Spark's
> laster major version (2.3.x) has netty 4.1
>
> I propose giving netty 4.1.x another shot.
>
> Sound good?
>
> Charles Allen
>