rymarm opened a new pull request, #2857:
URL: https://github.com/apache/drill/pull/2857

   # [DRILL-8467](https://issues.apache.org/jira/browse/DRILL-8467): Update 
netty to 4.1.101
   
   ## Description
   Update Netty to `4.1.101`. 
   Netty `4.1.75` has 2 "breaking" changes:
   1. The default `PooledByteBufAllocator` chunk size was reduced from `16 MiB` 
to `4 MiB`
   2. The default value of `io.netty.allocator.useCacheForAllThreads` was 
changed to `false`  
      
   Source: https://netty.io/news/2022/03/10/4-1-75-Final.html
   
   I made `InnerAllocator` creation with the previously accepted chunk size for 
Drill - 16 MiB, and left `useCacheForAllThreads` enabled because as far as I 
understand cache for all threads(not only for the Netty theads) gives better 
performance for Drill case. 
   
   I left those options overridable by Netty Java properties 
`io.netty.allocator.useCacheForAllThreads` and 
`io.netty.allocator.maxOrder`([responds for chunk 
size](https://github.com/netty/netty/blob/c29508672c7e9aa01bbe2a14f7438d82c5b25f91/buffer/src/main/java/io/netty/buffer/PooledByteBufAllocator.java#L104)),
 so it can be changed without Drill recompilation as it was before.
   
   ## Documentation
   No changes
   
   ## Testing
   Run unit tests.
   


-- 
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: dev-unsubscr...@drill.apache.org

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

Reply via email to