Skip to content

Remove unused code in PulsarAdminTool#12603

Closed
hezhangjian wants to merge 1 commit into
apache:masterfrom
hezhangjian:admin-tool-remove-unuse-code
Closed

Remove unused code in PulsarAdminTool#12603
hezhangjian wants to merge 1 commit into
apache:masterfrom
hezhangjian:admin-tool-remove-unuse-code

Conversation

@hezhangjian

Copy link
Copy Markdown
Member

Modifications

Remove unused code in PulsarAdminTool

Documentation

Check the box below and label this PR (if you have committer privilege).

Need to update docs?

  • no-need-doc

@github-actions

github-actions Bot commented Nov 3, 2021

Copy link
Copy Markdown

@shoothzj:Thanks for providing doc info!

@github-actions github-actions Bot added the doc-not-needed Your PR changes do not impact docs label Nov 3, 2021
@codelipenghui codelipenghui added this to the 2.10.0 milestone Nov 3, 2021
@yuruguo

yuruguo commented Nov 3, 2021

Copy link
Copy Markdown
Contributor

@codelipenghui @315157973 Please pay attention to this PR(#12581), there are differences between the two, remove or assign lastExitCode

@hezhangjian

Copy link
Copy Markdown
Member Author

@codelipenghui @315157973 Please pay attention to this PR(#12581), there are differences between the two, remove or assign lastExitCode

wow, #12581 is better

@hezhangjian hezhangjian closed this Nov 3, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

doc-not-needed Your PR changes do not impact docs

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants