Skip to content

Conversation

vesense
Copy link
Member

@vesense vesense commented Mar 24, 2017

https://issues.apache.org/jira/browse/ROCKETMQ-154

This is a very minor change. After execute command sh mqadmin, I get the last line info:
See 'mqadmin help <command>' for more information on a specific command.[xinwang@hadoop bin]$

Maybe we can improve this by adding a newline.

@coveralls
Copy link

Coverage Status

Coverage increased (+0.3%) to 31.798% when pulling d7cb174 on vesense:ROCKETMQ-154 into 15af63e on apache:develop.

2 similar comments
@coveralls
Copy link

Coverage Status

Coverage increased (+0.3%) to 31.798% when pulling d7cb174 on vesense:ROCKETMQ-154 into 15af63e on apache:develop.

@coveralls
Copy link

Coverage Status

Coverage increased (+0.3%) to 31.798% when pulling d7cb174 on vesense:ROCKETMQ-154 into 15af63e on apache:develop.

@vongosling
Copy link
Member

LGTM,please @shroman @lizhanhui help to review it~

@lizhanhui
Copy link
Contributor

lizhanhui commented Mar 27, 2017

+1, Thanks @vesense

@shroman
Copy link
Contributor

shroman commented Mar 28, 2017

+1 from me

asfgit pushed a commit that referenced this pull request Mar 28, 2017
Signed-off-by: shtykh_roman <rshtykh@yahoo.com>
@shroman
Copy link
Contributor

shroman commented Mar 28, 2017

Merged to develop branch. Thanks @vesense. Please close this PR.

@vesense
Copy link
Member Author

vesense commented Mar 28, 2017

Thanks & Closed.

@vesense vesense closed this Mar 28, 2017
asfgit pushed a commit that referenced this pull request Jun 6, 2017
Signed-off-by: shtykh_roman <rshtykh@yahoo.com>
lizhanhui pushed a commit to lizhanhui/rocketmq that referenced this pull request Jun 25, 2019
JiaMingLiu93 pushed a commit to JiaMingLiu93/rocketmq that referenced this pull request May 28, 2020
Signed-off-by: shtykh_roman <rshtykh@yahoo.com>
JiaMingLiu93 pushed a commit to JiaMingLiu93/rocketmq that referenced this pull request May 28, 2020
Signed-off-by: shtykh_roman <rshtykh@yahoo.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants