Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Initial commit for a go based pulsar cli. #2609

Open
wants to merge 6 commits into
base: master
from

Conversation

Projects
None yet
4 participants
@cckellogg
Copy link
Contributor

cckellogg commented Sep 19, 2018

Motivation

Initial patch to create a pulsar go based admin cli. This will help improve the cli performance, make the user experience better with more detailed examples and help documentation and simply the command structure.

Modifications

New go based admin cli

cckellogg added some commits Sep 19, 2018

@sijie

This comment has been minimized.

Copy link
Contributor

sijie commented Sep 19, 2018

@cckellogg you might need to run 'mvn license:format' to fix the license header issue.

@cckellogg

This comment has been minimized.

Copy link
Contributor Author

cckellogg commented Sep 19, 2018

@sijie does a license need to be added to the source files only? Do the mod files and readme need a license too?

@sijie

This comment has been minimized.

Copy link
Contributor

sijie commented Sep 19, 2018

@cckellogg ideally yes. but mvn license:format should already attaching the license headers.

@@ -0,0 +1,62 @@
//

This comment has been minimized.

Copy link
@kramasamy

kramasamy Sep 19, 2018

since the cluster create/delete/modify is used mainly for replication - it might be good to create a replication command group and put these underneath.

@sijie sijie modified the milestones: 2.3.0, 2.4.0 Dec 27, 2018

@sijie

This comment has been minimized.

Copy link
Contributor

sijie commented Dec 27, 2018

Move this to 2.4.0

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
You can’t perform that action at this time.