lianetm commented on PR #17440:
URL: https://github.com/apache/kafka/pull/17440#issuecomment-2499021858

   Regarding:
   
   > However, the test `testPatternSubscriptionWithNoTopicAccess` seems to have 
a different root cause. I’ve noticed that this test does not trigger the 
ConsumerRebalanceListenerCallbackCompletedEvent at all. It might be worth 
opening a new JIRA ticket to track this issue. WDYT?
   
   Agree that is unrelated to this PR. Looking into it, that test is actually 
checking a behviour that is different between the 2 protocols/consumer. With 
the new protocol a member must always provide subscribed topics (or regex) to 
join a group, so if subscribed to a java pattern where no topics are found 
(because of auth, or any reason), there won't be callbacks triggered. Seems to 
me that test should only run for the classic consumer (and maybe let's just add 
a comment about how the behaviour is different with the new protocol). Makes 
sense?


-- 
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: jira-unsubscr...@kafka.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org

Reply via email to