pulsar icon indicating copy to clipboard operation
pulsar copied to clipboard

[fix][broker] Fix replicator producers can not share with other exclusive producers

Open oneby-wang opened this issue 2 months ago • 1 comments

Fixes https://github.com/apache/pulsar/issues/24914

Motivation

Fix geo replication doesn't support normal exclusive producers.

Modifications

Modify AbstractTopic#incrementTopicEpochIfNeeded() method to support normal exclusive producers can share with replicator producers.

Verifying this change

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

This change is already covered by existing tests, such as (please describe tests).

Old test cases are covered by ExclusiveProducerTest test class.

New test method is OneWayReplicatorTest#testReplicatorProducerWithExclusiveAccessMode().

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

If the box was checked, please highlight the changes

  • [ ] Dependencies (add or upgrade a dependency)
  • [ ] The public API
  • [ ] The schema
  • [ ] The default values of configurations
  • [ ] The threading model
  • [ ] The binary protocol
  • [ ] The REST endpoints
  • [ ] The admin CLI options
  • [ ] The metrics
  • [ ] Anything that affects deployment

Documentation

  • [ ] doc
  • [ ] doc-required
  • [x] doc-not-needed
  • [ ] doc-complete

Matching PR in forked repository

PR in forked repository: https://github.com/oneby-wang/pulsar/pull/9

oneby-wang avatar Oct 31 '25 07:10 oneby-wang

Yes, I also noticed that, seems two contradictory aspects. I had posted a discussion mail to [email protected], please review: https://lists.apache.org/thread/ccr8p5wvhogljx7qwpf0hk4ort8j48nv.

oneby-wang avatar Oct 31 '25 14:10 oneby-wang