Skip to content

Comments

[Issue 3409] Support '--help' option of Pulsar command-line tools#3410

Merged
merlimat merged 9 commits intoapache:masterfrom
murong00:branch-3409
Feb 22, 2019
Merged

[Issue 3409] Support '--help' option of Pulsar command-line tools#3410
merlimat merged 9 commits intoapache:masterfrom
murong00:branch-3409

Conversation

@murong00
Copy link
Contributor

Motivation

Fixes #3409

Modifications

Support '--help' option of Pulsar command-line tools and modify usage of 'pulsar-perf'.

Copy link
Contributor

@merlimat merlimat left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

👍

Copy link
Contributor

@rdhabalia rdhabalia left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

👍

Environment variables:
PULSAR_LOG_CONF Log4j configuration file (default $DEFAULT_LOG_CONF)
PULSAR_CLIENT_CONF Configuration file for client (default: $DEFAULT_CLIENT_CONF)
PULSAR_CLIENT_CONF Configuration file for client (default: $DEFAULT_CLIENT_CONF)
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The indentation here is also fix by the way.
help

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Changed, thanks!

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Changed, thanks!

well done.

@ambition119
Copy link
Contributor

👍

@jiazhai
Copy link
Member

jiazhai commented Jan 24, 2019

retest this please

@sijie sijie added area/cli type/bug The PR fixed a bug or issue reported a bug labels Jan 25, 2019
@sijie
Copy link
Member

sijie commented Jan 25, 2019

retest this please

@sijie
Copy link
Member

sijie commented Jan 28, 2019

run cpp tests

4 similar comments
@murong00
Copy link
Contributor Author

run cpp tests

@merlimat
Copy link
Contributor

run cpp tests

@sijie
Copy link
Member

sijie commented Feb 2, 2019

run cpp tests

@merlimat
Copy link
Contributor

merlimat commented Feb 3, 2019

run cpp tests

@sijie
Copy link
Member

sijie commented Feb 6, 2019

run cpp tests

@sijie
Copy link
Member

sijie commented Feb 9, 2019

run cpp tests
run java8 tests

@sijie
Copy link
Member

sijie commented Feb 9, 2019

run cpp tests

1 similar comment
@merlimat
Copy link
Contributor

run cpp tests

@murong00
Copy link
Contributor Author

@merlimat @sijie The cpp tests failed due to a minor bug in pulsar-daemon, PTAL, thanks!

@merlimat merlimat added this to the 2.3.0 milestone Feb 14, 2019
@murong00
Copy link
Contributor Author

run java8 tests

@merlimat merlimat modified the milestones: 2.3.0, 2.3.1 Feb 14, 2019
@sijie
Copy link
Member

sijie commented Feb 15, 2019

run java8 tests

@merlimat merlimat merged commit c0ffa16 into apache:master Feb 22, 2019
merlimat pushed a commit that referenced this pull request Mar 29, 2019
)

* Support '--help' option of Pulsar command-line tools.

* Fix a indentation

* Fix a minor bug
@merlimat
Copy link
Contributor

merlimat commented Apr 1, 2019

Merged in 2.3.1 at 043f32f

@murong00 murong00 deleted the branch-3409 branch March 8, 2020 04:23
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

area/cli type/bug The PR fixed a bug or issue reported a bug

Projects

None yet

Development

Successfully merging this pull request may close these issues.

7 participants