lordcheng10 commented on PR #18633: URL: https://github.com/apache/pulsar/pull/18633#issuecomment-1331556351
> > @lordcheng10 it is very nice of you to explain this to me. i agree with you. it seems in MultiTopicConsumer the CompletableFuture subscription is merged among all partition. > > in my opinion if when one partition trigger exception logic, only remove the relevant consumer is more elegant. > > definity the current modification works, look forward to other people's idea. > > If a topic has a partition subscribe failure, then all partitions should fail, so all partitions of this topic should be removed from consumers. @WJL3333 IMO, it's fine to remove all partitions or just one partition -- 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]
