merlimat opened a new pull request #11952:
URL: https://github.com/apache/pulsar/pull/11952
### Motivation
`ReplicatorTest.testRemoveClusterFromNamespace()` is flaky because it's
trying to do:
1. Use `r4` as an alias cluster for `r3`
2. Create a topic with replication to clusters `r1`, `r3` and `r4`
3. Remove `r4` and check that the replicator got disconnected
The problem is that the broker will try to get 2 replicators (for `r3` and
`r4`) to the same broker instance, causing a "producer with same name is
already connected" error). Depending on if the `r3` or `r4` replicator connects
first, the test would succeed or fail.
### Modifications
* Pulled the test in dedicated class
* Use `r1`, `r2` and `r3` clusters, without aliasing
* Delete `r3` cluster to prove the point
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
