pulsar icon indicating copy to clipboard operation
pulsar copied to clipboard

[feat][broker]PIP-180 ShadowTopic - Part IV - Add Shadow Replicator

Open Jason918 opened this issue 2 years ago • 6 comments

Master Issue: https://github.com/apache/pulsar/issues/16153

Motivation

Add ShadowReplicator.

Modifications

  • Add map PersistentTopic#shadowReplicators, most of its management just follows PersistentTopic#replicators
  • Add class ShadowReplicator

Verifying this change

  • [x] Make sure that the change passes the CI checks.

This change added tests and can be verified as follows:

  • ShadowReplicatorTest

Does this pull request potentially affect one of the following parts:

None

Documentation

Check the box below or label this PR directly.

Need to update docs?

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

  • [x] doc-not-needed Internal implementation.

  • [ ] doc (Your PR contains doc changes)

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

Jason918 avatar Aug 31 '22 09:08 Jason918

/pulsarbot run-failure-checks

Jason918 avatar Sep 03 '22 07:09 Jason918

@AnonHxy Addressed comments. PTAL

Jason918 avatar Sep 06 '22 14:09 Jason918

LGTM now

AnonHxy avatar Sep 07 '22 05:09 AnonHxy

/pulsarbot run-failure-checks

Jason918 avatar Sep 07 '22 06:09 Jason918

@codelipenghui @eolivelli @Technoboy- PTAL

Jason918 avatar Sep 07 '22 06:09 Jason918

@codelipenghui PTAL

Jason918 avatar Sep 21 '22 04:09 Jason918