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

docs(cli):add restart command in pulsar-daemon #12373

Merged
merged 2 commits into from
Oct 26, 2021
Merged

docs(cli):add restart command in pulsar-daemon #12373

merged 2 commits into from
Oct 26, 2021

Conversation

ericsyh
Copy link
Contributor

@ericsyh ericsyh commented Oct 14, 2021

Signed-off-by: Eric Shen ericshenyuhao@outlook.com

(If this PR fixes a github issue, please add Fixes #<xyz>.)

Related PR: #12279

Motivation

Update the pulsar-daemon command docs since a new command added.

Modifications

Added the restart command in cli tools docs.

Documentation

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

Need to update docs?

  • doc-required

    (If you need help on updating docs, create a doc issue)

  • no-need-doc

    (Please explain why)

  • doc

    (If this PR contains doc changes)

Signed-off-by: Eric Shen <ericshenyuhao@outlook.com>
@Anonymitaet Anonymitaet added the doc Your PR contains doc changes, no matter whether the changes are in markdown or code files. label Oct 18, 2021
@Anonymitaet
Copy link
Member

Thanks for your contribution. Does this affect only master or other versioned docs?
If latter, could you please help update all affected versions? Thanks

Co-authored-by: Anonymitaet <50226895+Anonymitaet@users.noreply.github.com>
@ericsyh
Copy link
Contributor Author

ericsyh commented Oct 18, 2021

@Anonymitaet This change only affect master. Previously versions do not support this command.

@ericsyh ericsyh closed this Oct 24, 2021
@ericsyh ericsyh deleted the docs-pulsar-daemon-newcommand branch October 24, 2021 08:50
@ericsyh ericsyh restored the docs-pulsar-daemon-newcommand branch October 24, 2021 11:36
@ericsyh ericsyh reopened this Oct 24, 2021
@ericsyh
Copy link
Contributor Author

ericsyh commented Oct 25, 2021

@Anonymitaet i think this pr can be merged also.

@Anonymitaet Anonymitaet merged commit 23bb760 into apache:master Oct 26, 2021
zeo1995 pushed a commit to zeo1995/pulsar that referenced this pull request Oct 27, 2021
* up/master:
  fix delete authentication policies when delete topic. (apache#12215)
  [Broker] Fix messageDedup delete inactive producer name (apache#12493)
  Fixed getting children of parent nodes in LocalMemoryMetadataStore (apache#12491)
  Update Producer stats on producer close() (apache#12500)
  docs(cli):add restart command in pulsar-daemon (apache#12373)
  Add the pulsar java property memory allocator doc (apache#12481)
  [Doc]Update ci-documentbot.yml (apache#12480)
eolivelli pushed a commit to eolivelli/pulsar that referenced this pull request Nov 29, 2021
* docs(cli):add restart command in pulsar-daemon

Signed-off-by: Eric Shen <ericshenyuhao@outlook.com>

* Update site2/docs/reference-cli-tools.md

Co-authored-by: Anonymitaet <50226895+Anonymitaet@users.noreply.github.com>

Co-authored-by: Anonymitaet <50226895+Anonymitaet@users.noreply.github.com>
@ericsyh ericsyh deleted the docs-pulsar-daemon-newcommand branch December 19, 2021 12:42
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
doc Your PR contains doc changes, no matter whether the changes are in markdown or code files.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants