GitHub PushEvent at 2014-11-20 23:43:44
Repository: netty
Actor: Scottmitch
Payload: {"shas":[["9465db25bac6758a4c32aec73080c5937faddf9f","ecc73e9c8dcdd4253d165cf2a623dbb1b8ccacac@gmail.com","Small performance improvements\n\nMotivation:\n\nFound performance issues via FindBugs and PMD.\n\nModifications:\n\n- Removed unnecessary boxing/unboxing operations in DefaultTextHeaders.convertToInt(CharSequence) and DefaultTextHeaders.convertToLong(CharSequence). A boxed primitive is created from a string, just to extract the unboxed primitive value.\n- Added a static modifier for DefaultHttp2Connection.ParentChangedEvent class. This class is an inner class, but does not use its embedded reference to the object which created it. This reference makes the instances of the class larger, and may keep the reference to the creator object alive longer than necessary.\n- Added a static compiled Pattern to avoid compile it each time it is used when we need to replace some part of authority.\n- Improved using of StringBuilders.\n\nResult:\n\nPerformance improvements.","Idel Pivnitskiy",true],["2183c129d1b44e530c03624715492148002bfad4","8efd86fb78a56a5145ed7739dcb00c78581c5375@motd.kr","Use FastThreadLocal for skipFlagsCache\n\nRelated: #3113\n\nMotivation:\n\nBecause ChannelHandlerContext is most often instantiated by an I/O\nthread, we can rely on thread-local variables to keep the skipFlags\ncache, which should be faster than partitioned synchronized variable.\n\nModifications:\n\nUse FastThreadLocal for skipFlagsCache instead of partitioned\nsynchronized map.\n\nResult:\n\nLess contention","Trustin Lee",true],["000d3a55c5b771b7ed8b71f54b44de72a85306c0","ae63d1d4d7994483a3e520cafec7a829fe25c0a1@gmail.com","Adding codeAsText to HttpResponseStatus.\n\nMotivation:\nI found myself writing AsciiString constants in my code for\nresponse statuses and thought that perhaps it might be nice to have\nthem defined by Netty instead.\n\nModifications:\nAdding codeAsText to HttpResponseStatus that returns the status code as\nAsciiText.\n\nIn addition, added the 421 Misdirected Request response code from\nhttps://tools.ietf.org/html/draft-ietf-httpbis-http2-15#section-9.1.2\n\nThis response header was renamed in draft 15:\nhttps://tools.ietf.org/html/draft-ietf-httpbis-http2-15#appendix-A.1\nBut the code itself was not changed, and I thought using the latest would\nbe better.\n\nResult:\nIt is now possible to specify a status like this:\nnew DefaultHttp2Headers().status(HttpResponseStatus.OK.codeAsText());","Daniel Bevenius",true],["5f8483646b22f7618b967f1e48c220b0f798ec23","8efd86fb78a56a5145ed7739dcb00c78581c5375@motd.kr","Clean up 000d3a55c5b771b7ed8b71f54b44de72a85306c0\n\n- Rename httpResponseStatus() to newStatus()\n  - Move newStatus up so that static methods are grouped together\n- Rename codeAsString to codeAsText","Trustin Lee",true],["d7f5353fe8a0db785e13a622a40dfcf6b9e88802","8efd86fb78a56a5145ed7739dcb00c78581c5375@motd.kr","Fix memory leak in NameResolverGroup\n\nMotivation:\n\nNameResolverGroup uses the EventExecutor specified with getResolver() as\nthe key of its internal map.  Because the EventExecutor is often a\nwrapped one, such as PausibleChannelEventExecutor, which actually is a\nwrapper of the same executor, they should instantiate only one\nNameResolver.\n\nModifications:\n\nUnwrap the EventExecutor before using it as the key of the internal map\n\nResult:\n\nMemory leak is gone.","Trustin Lee",true],["22c8ee7d160ac4bbaa2acd72b7f31dfc31292b97","8efd86fb78a56a5145ed7739dcb00c78581c5375@motd.kr","Use Proxy-Authorization instead of Authorization for proxy authentication\n\nMotivation:\n\nWhen authenticating with a proxy server, HttpProxyHandler should use the\n'Proxy-Authorization' header rather than the 'Authorization' header.\n\nModifications:\n\n- Use 'Proxy-Authorization' header\n\nResult:\n\nCan connect to an HTTP proxy server","Trustin Lee",true],["6a30c9534b942df56b8a54e7fa3721246e58db32","ae63d1d4d7994483a3e520cafec7a829fe25c0a1@gmail.com","Adding a propagateSettings flag to InboundHttp2ToHttpAdapter.\n\nMotivation:\nWhen DefaultHttp2FrameReader has read a settings frame, the settings\nwill be passed along the pipeline. This allows a client to hold off\nsending data until it has received a settings frame. But for a server it\nwill always have received a settings frame and the usefulness of this\nforwarding of settings is less useful. This also causes a debug message\nto be logged on the server side if there is no channel handler to handle\nthe settings:\n\n[nioEventLoopGroup-1-1] DEBUG io.netty.channel.DefaultChannelPipeline -\nDiscarded inbound message {INITIAL_WINDOW_SIZE=131072,\nMAX_FRAME_SIZE=16384} that reached at the tail of the pipeline. Please\ncheck your pipeline configuration.\n\nModifications:\nAdded a builder for the InboundHttp2ToHttpAdapter and\nInboundHttp2PriortyAdapter and a new parameter named 'propagateSettings'\nto their constructors.\n\nResult:\nIt is now possible to control whether settings should be passed along\nthe pipeline or not.","Daniel Bevenius",true],["9173a2fb7c4ac9befbab5d8bb543c6e37d786a60","c817caf7d3f255e041fb7a494886d2ea5623b4a4@gmail.com","HTTP/2 Frame Listener Comment Correction\n\nMotivation:\nSome of the comments in HTTP/2 Frame Listener interface are misleading.\n\nModifications:\nClarify comments in Http2FrameListener.\n\nResult:\nHttp2FrameListener onDataRead comments are clarified.","Scott Mitchell",true],["908b68da03eb9b5f20ac5b49ae721f8a8d845f14","c817caf7d3f255e041fb7a494886d2ea5623b4a4@gmail.com","HTTP/2 Decompress/Compress buffer leak\n\nMotivation:\nThe interface for HTTP/2 onDataRead states that buffers will be released by the codec.  The decompressor and compressor methods are not releasing buffers created during the decompression/compression process.\n\nModifications:\nAfter onDataRead calls the decompressor and compressor classes will release the data buffer.\n\nResult:\nHTTP/2 compressor/decompressors are consistent with onDataRead interface assumptions.","Scott Mitchell",true]],"size":9,"ref":"refs/heads/master","head":"908b68da03eb9b5f20ac5b49ae721f8a8d845f14"}