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.

Sign up for GitHub

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

Jump to bottom

Support for setting geo-replication clusters on topic level #12136

Merged
merged 5 commits into from
Nov 4, 2021
Merged

Support for setting geo-replication clusters on topic level #12136

merged 5 commits into from
Nov 4, 2021

Conversation

Technoboy-
Copy link
Contributor

@Technoboy- Technoboy- commented Sep 22, 2021 •
edited by codelipenghui
Loading

Motivation

Currently, when we try to enable the geo-replication, we need to set multiple clusters for the namespaces, and all the topics under the namespace will start to replicate data across clusters.

It's useful for adding the replicate clusters in the topic level so that we can achieve the topics under a namespace can have different replication clusters such as topic A(cluster-a <-> cluster-b) topic B(cluster-b <-> cluster-c).

Documentation

I'm not sure if the doc could generate automatically. If no, I will update the doc later.

@Anonymitaet
Copy link
Member

I'm not sure if the doc could generate automatically. If no, I will update the doc later.

@Technoboy- the doc you added can be generated automatically.

@Anonymitaet Anonymitaet added the doc Your PR contains doc changes, no matter whether the changes are in markdown or code files. label Sep 23, 2021
@codelipenghui codelipenghui added this to the 2.9.0 milestone Sep 23, 2021
@Technoboy-
Copy link
Contributor Author

/pulsarbot run-failure-checks

3 similar comments
@Technoboy-
Copy link
Contributor Author

/pulsarbot run-failure-checks

@Technoboy-
Copy link
Contributor Author

/pulsarbot run-failure-checks

@Technoboy-
Copy link
Contributor Author

/pulsarbot run-failure-checks

@315157973
Copy link
Contributor

@hangc0276 Please review again

@eolivelli eolivelli modified the milestones: 2.9.0, 2.10.0 Oct 6, 2021
@Anonymitaet
Copy link
Member

Anonymitaet commented Oct 28, 2021 •
edited
Loading

@Technoboy-

  1. Do we need to add docs here? Like adding a section "Set geo-replication clusters on a topic level", and adding explanations, commands, and examples.

  2. Do we need to update docs as below?
    https://pulsar.apache.org/docs/en/next/administration-geo/#geo-replication-and-pulsar-properties
    image

@Technoboy-
Copy link
Contributor Author

@Technoboy- do we need to add docs here? Like adding a section "Set geo-replication clusters on a topic level", and adding explanations, commands, and examples.

Seem that the doc could be generated automatically.

@Technoboy-
Copy link
Contributor Author

@Technoboy-

  1. Do we need to add docs here? Like adding a section "Set geo-replication clusters on a topic level", and adding explanations, commands, and examples.
  2. Do we need to update docs as below?
    https://pulsar.apache.org/docs/en/next/administration-geo/#geo-replication-and-pulsar-properties
    image

Yes, right. I will update this part later.

@Technoboy-
Copy link
Contributor Author

/pulsarbot run-failure-checks

@315157973
Copy link
Contributor

/pulsarbot run-failure-checks

@hangc0276 hangc0276 merged commit 04ddc9b into apache:master Nov 4, 2021
@Anonymitaet Anonymitaet mentioned this pull request Nov 5, 2021
Merged
@codelipenghui codelipenghui added type/feature The PR added a new feature or issue requested a new feature release/2.8.3 release/2.9.1 type/enhancement The enhancements for the existing features or docs. e.g. reduce memory usage of the delayed messages and removed release/2.8.2 release/2.9.1 type/feature The PR added a new feature or issue requested a new feature labels Nov 5, 2021
eolivelli pushed a commit to eolivelli/pulsar that referenced this pull request Nov 29, 2021
@Technoboy- @eolivelli
Support for setting geo-replication clusters on topic level (apache#1…
9af6d06 
…2136)

### Motivation
Currently, when we try to enable the geo-replication, we need to set multiple clusters for the namespaces, and all the topics under the namespace will start to replicate data across clusters.

It's useful for adding the replicate clusters in the topic level so that we can achieve the topics under a namespace can have different replication clusters such as topic A(cluster-a <-> cluster-b) topic B(cluster-b <-> cluster-c).

### Documentation
I'm not sure if the doc could generate automatically. If no, I will update the doc later.
@codelipenghui codelipenghui mentioned this pull request Mar 15, 2022
Closed
@Technoboy- Technoboy- deleted the support-topic-policies-rep branch August 10, 2022 05:54
@rdhabalia rdhabalia mentioned this pull request Nov 30, 2022
Merged
13 tasks
@liangyepianzhou liangyepianzhou mentioned this pull request Nov 13, 2023
Closed
15 tasks
@liangyepianzhou liangyepianzhou mentioned this pull request Apr 10, 2024
Merged
15 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@hangc0276 hangc0276 hangc0276 approved these changes

@315157973 315157973 315157973 approved these changes

@codelipenghui codelipenghui codelipenghui approved these changes

@merlimat merlimat Awaiting requested review from merlimat

@eolivelli eolivelli Awaiting requested review from eolivelli

@sijie sijie Awaiting requested review from sijie

@gaoran10 gaoran10 Awaiting requested review from gaoran10

Assignees

@Technoboy- Technoboy-

Labels
doc Your PR contains doc changes, no matter whether the changes are in markdown or code files. type/enhancement The enhancements for the existing features or docs. e.g. reduce memory usage of the delayed messages
Projects
None yet
Milestone
2.10.0
Development

Successfully merging this pull request may close these issues.

None yet
6 participants
@Technoboy- @Anonymitaet @315157973 @hangc0276 @codelipenghui @eolivelli