Github user vongosling commented on the issue:
https://github.com/apache/incubator-rocketmq/pull/100
That's a good polish, but I must point out the 2 point for this
optimization. 1. please comply with PR best practice for title as you have
corrected before. 2. you can consider the
Github user coveralls commented on the issue:
https://github.com/apache/incubator-rocketmq/pull/100
[](https://coveralls.io/builds/11287015)
Coverage decreased (-0.09%) to 37.876% when pulling
Github user coveralls commented on the issue:
https://github.com/apache/incubator-rocketmq/pull/100
[](https://coveralls.io/builds/11287015)
Coverage decreased (-0.09%) to 37.876% when pulling
Github user coveralls commented on the issue:
https://github.com/apache/incubator-rocketmq/pull/100
[](https://coveralls.io/builds/11283185)
Coverage decreased (-0.06%) to 37.906% when pulling
Github user coveralls commented on the issue:
https://github.com/apache/incubator-rocketmq/pull/100
[](https://coveralls.io/builds/11283185)
Coverage decreased (-0.06%) to 37.906% when pulling
Github user coveralls commented on the issue:
https://github.com/apache/incubator-rocketmq/pull/100
[](https://coveralls.io/builds/11283185)
Coverage decreased (-0.06%) to 37.906% when pulling
