pulsar icon indicating copy to clipboard operation
pulsar copied to clipboard

[improve][admin] Add new flag `--no-ledger` to stats-internal command

Open gaozhangmin opened this issue 3 years ago • 1 comments

Motivation

Admin command stats-internal support hiding ledgers list.

If there are thousands of ledgers, it's meaningless to print so much ledgers.

In additional, partitioned-stats-internal also support this feature.

Modifications

Add new command line parameter -nl --no-ledger, default to false, which means printing ledgers list default.

Documentation

  • [ ] doc-required (Your PR needs to update docs and you will update later)

  • [x] doc-not-needed (Please explain why)

  • [ ] doc (Your PR contains doc changes)

  • [ ] doc-complete (Docs have been already added)

Matching PR in forked repository

PR in forked repository: <!-- ENTER URL HERE

After opening this PR, the build in apache/pulsar will fail and instructions will be provided for opening a PR in the PR author's forked repository.

apache/pulsar pull requests should be first tested in your own fork since the apache/pulsar CI based on GitHub Actions has constrained resources and quota. GitHub Actions provides separate quota for pull requests that are executed in a forked repository.

The tests will be run in the forked repository until all PR review comments have been handled, the tests pass and the PR is approved by a reviewer.

-->

gaozhangmin avatar Sep 21 '22 09:09 gaozhangmin

@gaozhangmin Please provide a correct documentation label for your PR. Instructions see Pulsar Documentation Label Guide.

github-actions[bot] avatar Sep 21 '22 10:09 github-actions[bot]

The pr had no activity for 30 days, mark with Stale label.

github-actions[bot] avatar Oct 22 '22 02:10 github-actions[bot]