[GitHub] [druid] xvrl commented on pull request #12032: Migrate Druid HttpClient from Netty 3 to 4

[GitBox <gi...@apache.org>]

xvrl commented on PR #12032:
URL: https://github.com/apache/druid/pull/12032#issuecomment-1251607235

   @liam-verta I think we're pretty close – barring merging changes from master – but one thing we're trying to understand is the change in memory requirements. It seems they are significantly higher with Netty 4. I was able to get the the failing query tests to pass by using `-Dio.netty.allocator.type=unpooled`. So far I have not seen any evidence of a leak: using `-Dio.netty.leakDetection.level=paranoid` doesn't seem to turn up anything so far. So it is possible the netty memory pooling algorithm requires significantly more memory, but I would like to make sure we understand why.


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: commits-unsubscribe@druid.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org


---------------------------------------------------------------------
To unsubscribe, e-mail: commits-unsubscribe@druid.apache.org
For additional commands, e-mail: commits-help@druid.apache.org