Skip to content

[doc] Improve Pulsar multi-cluster deployment#5167

Merged
sijie merged 3 commits intoapache:masterfrom
murong00:branch-5167
Sep 17, 2019
Merged

[doc] Improve Pulsar multi-cluster deployment#5167
sijie merged 3 commits intoapache:masterfrom
murong00:branch-5167

Conversation

@murong00
Copy link
Contributor

Motivation

Minor improvement of deploy-bare-metal-multi-cluster.md.

@sijie
Copy link
Member

sijie commented Sep 10, 2019

@Monica-zy since you were also updating this documentation, can you please help review this change?

@sijie sijie added the doc Your PR contains doc changes, no matter whether the changes are in markdown or code files. label Sep 10, 2019
@sijie sijie added this to the 2.5.0 milestone Sep 10, 2019
@Jennifer88huang-zz
Copy link
Contributor

Jennifer88huang-zz commented Sep 11, 2019

@Monica-zy There are some conflicts in this PR with yours in #5135. Maybe we can deal with this one first, and then you can continue with your updates in #5135.

Copy link
Contributor

@Jennifer88huang-zz Jennifer88huang-zz left a comment

Choose a reason for hiding this comment

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

Since @Monica-zy and @Anonymitaet are refining the language part of this topic in another PR(#5135), I've just refined contents related to what @murong00 has updated.
I've checked the modified language and links in this PR, there is no issue with those updates.

@Jennifer88huang-zz
Copy link
Contributor

run integration tests
run java8 tests

@sijie sijie merged commit c8ddbdd into apache:master Sep 17, 2019
@murong00 murong00 deleted the branch-5167 branch March 8, 2020 04:39
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.

3 participants

Comments