You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@pulsar.apache.org by GitBox <gi...@apache.org> on 2022/07/27 04:01:22 UTC
[GitHub] [pulsar] Technoboy- opened a new pull request, #16809: Fix Flaky-test: PartitionedProducerConsumerTest.testPartitionedTopicInterceptor
Technoboy- opened a new pull request, #16809:
URL: https://github.com/apache/pulsar/pull/16809
Fixes #16710
Master Issue: #16710
### Motivation
When the partition updates, the internal consumer may not connect to the broker(you can see `onPartitionsChange ` is before `startReceivingMessages`), so `unsubscribe` will throw the exception.
https://github.com/apache/pulsar/blob/bdd87794a0ad6a775a911e9cb27cf6fbaf02d9e8/pulsar-client/src/main/java/org/apache/pulsar/client/impl/MultiTopicsConsumerImpl.java#L1415-L1425
PartitionedProducerConsumerTest#testPartitionedTopicInterceptor tests for updating the partition, another way to fix is that deleting `consumer.unsubscribe()` at line 1029.
### Documentation
- [x] `doc-not-needed`
(Please explain why)
--
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: commits-unsubscribe@pulsar.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [pulsar] Technoboy- merged pull request #16809: [fix][tests] Fix Flaky-test: PartitionedProducerConsumerTest.testPartitionedTopicInterceptor
Posted by GitBox <gi...@apache.org>.
Technoboy- merged PR #16809:
URL: https://github.com/apache/pulsar/pull/16809
--
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: commits-unsubscribe@pulsar.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org