jack2012aa commented on PR #20982:
URL: https://github.com/apache/kafka/pull/20982#issuecomment-3573542826

   > Thanks for the PR @jack2012aa.
   > 
   > I'm a little concerned that switching to test only the `CLASSIC` group 
protocol could hide the bug mentioned in the PR description, namely that use of 
the `CONSUMER` group protocol could result in "duplicate messages" in this case.
   > 
   > Thanks.
   
   Thanks for the review @kirktrue! Apologize for my phrasing. In the 
description I mean that using the `CLASSIC` group protocol could lead to 
duplicate messages. `PARTITION_ASSIGNMENT_STRATEGY_CONFIG` is invalid when 
using the `CONSUMER` group protocol, so I removed it from the test, see 
[KAFKA-17338](https://issues.apache.org/jira/browse/KAFKA-17338).
   
   I will rephrase the description to make it clearer.


-- 
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]

Reply via email to