Merge pull request AsyncHttpClient#903 from sschepens/pooled_allocators

Use PooledByteByfAllocator by default

Author: slandelle

providers/netty4/src/main/java/org/asynchttpclient/netty/channel/ChannelManager.java

@@ -18,6 +18,7 @@
 import static org.asynchttpclient.util.HttpUtils.isWebSocket;
 import static org.asynchttpclient.util.MiscUtils.buildStaticIOException;
 import io.netty.bootstrap.Bootstrap;
+import io.netty.buffer.PooledByteBufAllocator;
 import io.netty.channel.Channel;
 import io.netty.channel.ChannelInitializer;
 import io.netty.channel.ChannelOption;
@@ -180,6 +181,12 @@ public Semaphore apply(Object partitionKey) {
         webSocketBootstrap = new Bootstrap().channel(socketChannelClass).group(eventLoopGroup);
         secureWebSocketBootstrap = new Bootstrap().channel(socketChannelClass).group(eventLoopGroup);
 
+        // default to PooledByteBufAllocator
+        plainBootstrap.option(ChannelOption.ALLOCATOR, PooledByteBufAllocator.DEFAULT);
+        secureBootstrap.option(ChannelOption.ALLOCATOR, PooledByteBufAllocator.DEFAULT);
+        webSocketBootstrap.option(ChannelOption.ALLOCATOR, PooledByteBufAllocator.DEFAULT);
+        secureWebSocketBootstrap.option(ChannelOption.ALLOCATOR, PooledByteBufAllocator.DEFAULT);
+
         if (config.getConnectTimeout() > 0)
             nettyConfig.addChannelOption(ChannelOption.CONNECT_TIMEOUT_MILLIS, config.getConnectTimeout());
         for (Entry<ChannelOption<Object>, Object> entry : nettyConfig.propertiesSet()) {