You are viewing a plain text version of this content. The canonical link for it is here.
Posted to issues@rocketmq.apache.org by "ASF GitHub Bot (JIRA)" <ji...@apache.org> on 2017/01/05 11:04:58 UTC
[jira] [Commented] (ROCKETMQ-11) Improve concision - Reuse local
variable 'brokerAddrs' in RouteInfoManager.getSystemTopicList method
[ https://issues.apache.org/jira/browse/ROCKETMQ-11?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15801070#comment-15801070 ]
ASF GitHub Bot commented on ROCKETMQ-11:
----------------------------------------
Github user naughtybear commented on the issue:
https://github.com/apache/incubator-rocketmq/pull/8
@vongosling
Hi,
two places still not align with the pr procedure.
1. pull request title should be "[ROCKETMQ-11] Improve concision - Reuse local variable 'brokerAddrs' in RouteInfoManager.getSystemTopicList method #8"
2. commit comment should be "[ROCKETMQ-11] Improve concision - Reuse local variable 'brokerAddrs' in RouteInfoManager.getSystemTopicList method, close apache/incubator-rocketmq#8"
Am I right? If so, I need start a new pull request and modify JIRA info to keep aligned.
Many thanks.
> Improve concision - Reuse local variable 'brokerAddrs' in RouteInfoManager.getSystemTopicList method
> ----------------------------------------------------------------------------------------------------
>
> Key: ROCKETMQ-11
> URL: https://issues.apache.org/jira/browse/ROCKETMQ-11
> Project: Apache RocketMQ
> Issue Type: Improvement
> Reporter: hanzhaozhan
> Assignee: vongosling
> Priority: Trivial
> Labels: concision
>
> Tiny improvement to make code concise.
> Firstly I thought it's for concurrency consideration. Then I realize it's not.
--
This message was sent by Atlassian JIRA
(v6.3.4#6332)