{"push_id":1475199540,"size":15,"distinct_size":15,"ref":"refs/heads/feature/issue-259","head":"ffb4f886c84d8e60471c29cbcf33ebe00d8c481c","before":"6d510a5327744a058ec585ed8c5fa90b609fecaa","commits":[{"sha":"a5ae265416637983921a739e17f9fcfedbaf6c12","author":{"name":"Masahiro Ide","email":"c327ff5dcc26623cb64883551d5749d0674d9f44@gmail.com"},"message":"Add metrics for HttpHealthCheckedEndpointGroup (#346)","distinct":true,"url":"https://api.github.com/repos/dongjinleekr/armeria/commits/a5ae265416637983921a739e17f9fcfedbaf6c12"},{"sha":"820b5b871c257a649e5da626ad16a91f0dba2462","author":{"name":"Trustin Lee","email":"213c4738cc35c04f021cb47d03c4226bdf330636@motd.kr"},"message":"Revamp RequestLog API (#343)\n\n* Revamp RequestLog API\r\n\r\nMotivation:\r\n\r\nRequestLog API currently notifies the log consumer only when a Request\r\nor a Response is fully processed. This does not work well with a\r\nstreaming request that can take potentially long time when the consumer\r\nwants to get certain information sooner.\r\n\r\nA good example would be the implementation of 'activeRequest' counter.\r\nDropwizardMetricConsumer increments the 'activeRequest' counter in\r\nonRequest() handler method and decrements in onResponse(). This is OK\r\nfor Thrift calls where request processing takes small amount of time,\r\nbut not OK for other streaming calls such as file upload.\r\n\r\nThere should be a way to let a log consumer tell the API what properties\r\nthe log consumer is interested in and RequestLog should notify the\r\nconsumer as soon as the interested properties are available.\r\n\r\nModifications:\r\n\r\n- Rewrite RequestLog and RequestLogBuilder\r\n  - ResponseLog and ResponseLogBuilder are merged into RequestLog and\r\n    RequestLogBuilder for simplicity\r\n  - Add RequestLogAvailability to define the availability of properties\r\n- Remove LogCollecting{Client,Service} because it doesn't do anything\r\n  useful anymore with the new API\r\n- Rename AbstractRpc{Request,Response} to DefaultRpc{Request,Response}\r\n- Remove ThriftCall and ThriftReply and use DefaultRpc{Request,Response}\r\n  - There was no reason to introduce these classes from the beginning...\r\n- Rename ApacheThrift{Call,Reply} to Thrift{Call,Reply}\r\n- Add the documentation about the new API\r\n  - Update CSS a little bit to make things look better\r\n  - Add a placeholder page for @kawamuray's Kafka structured logging\r\n  - Do not use Java classes in conf.py so that the site can be generated\r\n    with CPython\r\n- Miscellaneous:\r\n  - Make the formatter methods in TextFormatter append the raw value\r\n    when the value is formatted into an approximated (or human-friendly)\r\n    form\r\n\r\nResult:\r\n\r\n- Possible to get the necessary information sooner\r\n- Cannot add an attribute to RequestLog anymore\r\n  - May be both pro and con; will revisit if there's demand\r\n- RequestLog API now has documentation.\r\n\r\n* Follow-up commit\r\n\r\n- Do not set requestContent to null implicitly on serializationFormat(NONE)\r\n- Make AbstractHttpService set requestContent and serializationFormat\r\n  automatically\r\n- Add RequestLogBuilder.is{Request,Response}ContentDeferred()\r\n- Update documentation","distinct":true,"url":"https://api.github.com/repos/dongjinleekr/armeria/commits/820b5b871c257a649e5da626ad16a91f0dba2462"},{"sha":"fa737dbfff7301b967c95891080cef3ad6c69944","author":{"name":"Masahiro Ide","email":"c327ff5dcc26623cb64883551d5749d0674d9f44@gmail.com"},"message":"Add ThriftCompletableFuture and ThriftListenableFuture (#347)","distinct":true,"url":"https://api.github.com/repos/dongjinleekr/armeria/commits/fa737dbfff7301b967c95891080cef3ad6c69944"},{"sha":"0a0f6146b681ca162c470aade3e49f654bbde3a1","author":{"name":"Trustin Lee","email":"213c4738cc35c04f021cb47d03c4226bdf330636@motd.kr"},"message":"Update CONTRIBUTING.md (#344)\n\n- Update IntelliJ IDEA settings\r\n  - Use the 'import settings from JAR' feature for simplicity\r\n- Add how to build to README.md\r\n- Add the link to CONTRIBUTING.md to README.md\r\n- Add how to configure the project\r\n- Add the checklist based on @dongjinleekr's checklist:\r\n  - https://github.com/line/armeria/pull/328#issuecomment-263481827","distinct":true,"url":"https://api.github.com/repos/dongjinleekr/armeria/commits/0a0f6146b681ca162c470aade3e49f654bbde3a1"},{"sha":"6d3547a4118fc81cd7c852685f528ab9e95f865c","author":{"name":"Trustin Lee","email":"213c4738cc35c04f021cb47d03c4226bdf330636@motd.kr"},"message":"Update setup-gradle.rst and setup-maven.rst (#345)\n\n.. because it's outdated.\r\n\r\n- Remove the configuration related with Jetty ALPN agent\r\n  - We support Windows, Linux and OS X using netty-tcnative-boringssl-static.\r\n- List all module dependencies","distinct":true,"url":"https://api.github.com/repos/dongjinleekr/armeria/commits/6d3547a4118fc81cd7c852685f528ab9e95f865c"},{"sha":"6a76833b6a8fa1e2dd2a1b4b296c741ef1c5c4c7","author":{"name":"Trustin Lee","email":"213c4738cc35c04f021cb47d03c4226bdf330636@motd.kr"},"message":"Update the project version to 0.33.1-SNAPSHOT","distinct":true,"url":"https://api.github.com/repos/dongjinleekr/armeria/commits/6a76833b6a8fa1e2dd2a1b4b296c741ef1c5c4c7"},{"sha":"8acbaf2933648a7dd3982f6b18e46046f94c9683","author":{"name":"Anuraag Agrawal","email":"7580e1fcff05b113a6792a72e76190b7ff4bba74@gmail.com"},"message":"Update to grpc 1.0.2 (#349)\n\n- Now that grpc uses the same netty version as armeria, the interop test can be reenabled (usually there is no version conflict between grpc and armeria because the client and server would run in different JVM processes, but the interop test has the limit that it's all in the same JVM).\r\n\r\n- Implement new abstract method (will check with grpc if it was intended for this not to be implemented in AbstractServerStream I doubt it. We'd be able to remove in 1.0.3 if so.)\r\n\r\n- Update test with new stream listener calling conventions.","distinct":true,"url":"https://api.github.com/repos/dongjinleekr/armeria/commits/8acbaf2933648a7dd3982f6b18e46046f94c9683"},{"sha":"dc586a799fc06ef7c294c5fd2d85c1a05224ddf2","author":{"name":"Trustin Lee","email":"213c4738cc35c04f021cb47d03c4226bdf330636@motd.kr"},"message":"Do not check null on RpcRequest parameters / Easier instantiation of RpcRequest and RpcResponse (#352)\n\nRelated issue: #350\r\n\r\nMotivation:\r\n\r\n- Attempting to send a null parameter fails since 0.33.0 due to the use\r\n  of ImmutableList.copyOf() and GuavaCollectors.toImmutableList(), which\r\n  does numm checks on elements.\r\n- We could make users' lives easier by adding some static factory\r\n  methods to RpcRequest and RpcResponse\r\n\r\nModifications:\r\n\r\n- Do not use ImmutableList.copyOf() and\r\n  GuavaCollectors.toImmutableList() when constructing the RPC parameter\r\n  list\r\n- Add various static factory methods to RpcRequest and RpcResponse\r\n- Add more convenience constructors to DefaultRpcRequest\r\n\r\nResult:\r\n\r\n- Fixes #350, the regression introduced in 0.33.0\r\n- A user can instantiate RpcRequest and RpcResponse more easily.","distinct":true,"url":"https://api.github.com/repos/dongjinleekr/armeria/commits/dc586a799fc06ef7c294c5fd2d85c1a05224ddf2"},{"sha":"676c252ab46cdd8a182434f3e768cc988b36c441","author":{"name":"Masahiro Ide","email":"c327ff5dcc26623cb64883551d5749d0674d9f44@gmail.com"},"message":"Dedup endpoint health check metric and converts healthcheck metrics t… (#351)\n\nMotivation:\r\n\r\nWhen a user creates a HealthCheckedEndpointGroup with duplicated\r\nendpoint list, it throws IllegalStateException because it registers\r\nduplicated metric to MetricRegistry.\r\n\r\nModifications:\r\n- Add EndpointHealthStateGaugeSet\r\n- Expose number of servers registered to endpoing group,\r\n  healthy/unhealthy servers, and healthy/unhealthy server sets as Gauge metrics","distinct":true,"url":"https://api.github.com/repos/dongjinleekr/armeria/commits/676c252ab46cdd8a182434f3e768cc988b36c441"},{"sha":"55fe99402c6b6b188bd982132be9e60d748c76f5","author":{"name":"Trustin Lee","email":"040b30789c62a3676929a311f5b2b3182d3a4c09@linecorp.com"},"message":"Update the project version to 0.34.0-SNAPSHOT","distinct":true,"url":"https://api.github.com/repos/dongjinleekr/armeria/commits/55fe99402c6b6b188bd982132be9e60d748c76f5"},{"sha":"586134ba65572ba9a392531dc5449955d9609628","author":{"name":"Masahiro Ide","email":"c327ff5dcc26623cb64883551d5749d0674d9f44@gmail.com"},"message":"Fix StaticEndpointGroupTest to allocate the server port dynamically (#353)","distinct":true,"url":"https://api.github.com/repos/dongjinleekr/armeria/commits/586134ba65572ba9a392531dc5449955d9609628"},{"sha":"e3257ddcc9cb2c97591ac184b565ba94ee6edfdf","author":{"name":"Koji Lin","email":"e143ebb84e48ed1b41af953a4bb9420d1c696faf@gmail.com"},"message":"Avoid creating humongous object when armeria retrofit get large response (#354)\n\nMotivation:\r\n\r\n- Original armeria-retrofit will convert huge response body to single byte[].\r\n\r\nModifications:\r\n\r\n- Using subscriber to add response data to okio.Buffer which creates multiple.\r\n\r\nResult:\r\n\r\n- Avoid humongous object creation which is not good for memory management.","distinct":true,"url":"https://api.github.com/repos/dongjinleekr/armeria/commits/e3257ddcc9cb2c97591ac184b565ba94ee6edfdf"},{"sha":"0824cb738e0443fa320e8bc7dd5221b0d881f88f","author":{"name":"Masahiro Ide","email":"c327ff5dcc26623cb64883551d5749d0674d9f44@gmail.com"},"message":"Make the number of boss I/O threads configurable (#356)\n\nFixes #348","distinct":true,"url":"https://api.github.com/repos/dongjinleekr/armeria/commits/0824cb738e0443fa320e8bc7dd5221b0d881f88f"},{"sha":"5d61db1cdfd14dc4faa2a945a826d8df2714aad1","author":{"name":"Masahiro Ide","email":"c327ff5dcc26623cb64883551d5749d0674d9f44@gmail.com"},"message":"Add ServerBuilder#serverListener (#358)","distinct":true,"url":"https://api.github.com/repos/dongjinleekr/armeria/commits/5d61db1cdfd14dc4faa2a945a826d8df2714aad1"},{"sha":"ffb4f886c84d8e60471c29cbcf33ebe00d8c481c","author":{"name":"Lee Dongjin","email":"9ba935863f5cede19c295a42896f90eae592e6a8@apache.org"},"message":"Implement HTTP authorization decorator service (#259)","distinct":true,"url":"https://api.github.com/repos/dongjinleekr/armeria/commits/ffb4f886c84d8e60471c29cbcf33ebe00d8c481c"}]}
{"actor":{"display_login":"dongjinleekr"}}