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

Add multi-hosts example for bookkeeperMetadataServiceUri #6998

Merged
merged 2 commits into from Jun 4, 2020

Conversation

murong00
Copy link
Contributor

Motivation

Some users may be confused by bookkeeperMetadataServiceUri in multi-hosts case (e.g. comma list like zk+hierarchical://zk1:2181,zk2:2181/ledgers) since bookkeeper only support semicolon separated values currently ( my colleague is tracing this in bookkeeper master, we can overcome this in future).

Modifications

Add multi-hosts usage example in broker.conf.

Copy link
Contributor

@Huanli-Meng Huanli-Meng left a comment

Choose a reason for hiding this comment

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

Leave a comments, PTAL.

site2/docs/reference-configuration.md Outdated Show resolved Hide resolved
@sijie sijie added this to the 2.6.0 milestone May 22, 2020
@sijie sijie added the doc Your PR contains doc changes, no matter whether the changes are in markdown or code files. label May 22, 2020
@codelipenghui
Copy link
Contributor

@murong00 Would you please rebase the master branch since there are some new changes related to the failed CI tests.

@murong00
Copy link
Contributor Author

/pulsarbot run-failure-checks

@sijie
Copy link
Member

sijie commented Jun 2, 2020

/pulsarbot run-failure-checks

2 similar comments
@codelipenghui
Copy link
Contributor

/pulsarbot run-failure-checks

@murong00
Copy link
Contributor Author

murong00 commented Jun 3, 2020

/pulsarbot run-failure-checks

@codelipenghui codelipenghui merged commit 3980b37 into apache:master Jun 4, 2020
zeo1995 pushed a commit to zeo1995/pulsar that referenced this pull request Jun 5, 2020
…te-update

* 'website-update' of github.com:zeo1995/pulsar: (432 commits)
  Fixed ordering issue in KeyShared dispatcher when adding consumer (apache#7106)
  Fix Duplicated messages are sent to dead letter topic apache#6960 (apache#7021)
  [Issue 2793][Doc]--Update the TLS hostname verification for CPP and Python clients (apache#7162)
  [Doc]--set netty mex frame size (apache#7174)
  [Doc] Update for the maximum message size (apache#7171)
  Fixed KeyShared consumers getting stuck on delivery (apache#7105)
  [apache#6003][pulsar-functions] Possibility to add builtin Functions (apache#6895)
  [Issue 6921][pulsar-broker-common] Replaced "Paths.get(...).getParent()", because it's system dependent and uses '\' as path separator on Windows (apache#6992)
  Improve broker unit test CI (apache#7173)
  Fix typo in exception message (apache#7027)
  Support KeyValue Schema Use Null Key And Null Value (apache#7139)
  [Doc]--Update documents for support consumer priority level in failover mode (apache#7136)
  Add schema config to cpp and cgo docs. (apache#7137)
  [Doc]--Update for the maximum message size (apache#7160)
  [C++] Expose ZSTD and Snappy compression to C API (apache#7014)
  [pulsar-proxy] add proxyLogLevel into config file (apache#6948)
  Add multi-hosts example for bookkeeperMetadataServiceUri (apache#6998)
  support for termination of partitioned topic (apache#6126)
  Use pure-java Air-Compressor instead of JNI based libraries (apache#5390)
  [Issues 5709]remove the namespace checking (apache#5716)
  ...

# Conflicts:
#	site2/website/scripts/split-swagger-by-version.js
cdbartholomew pushed a commit to kafkaesque-io/pulsar that referenced this pull request Jul 24, 2020
Motivation
Some users may be confused by bookkeeperMetadataServiceUri in multi-hosts case (e.g. comma list like zk+hierarchical://zk1:2181,zk2:2181/ledgers) since bookkeeper only support semicolon separated values currently ( my colleague is tracing this in bookkeeper master, we can overcome this in future).

Modifications
Add multi-hosts usage example in broker.conf.
huangdx0726 pushed a commit to huangdx0726/pulsar that referenced this pull request Aug 24, 2020
Motivation
Some users may be confused by bookkeeperMetadataServiceUri in multi-hosts case (e.g. comma list like zk+hierarchical://zk1:2181,zk2:2181/ledgers) since bookkeeper only support semicolon separated values currently ( my colleague is tracing this in bookkeeper master, we can overcome this in future).

Modifications
Add multi-hosts usage example in broker.conf.
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.

None yet

5 participants