chenyulin0719 commented on PR #18513: URL: https://github.com/apache/kafka/pull/18513#issuecomment-2647140935
Hi @dajac, @chia7712 , Sorry for the late update. I've done the investigations and update a newer version. Also added my feedback under each discussion. The latest changes are: 1. Extract member removal to new test: `testConsumerGroupWithMemberRemoval` 2. Move group id argument from `BackgroundConsumerSet`, set it externally. You can run below command to validate the change: ``` N=100; I=0; while [ $I -lt $N ] && ./gradlew core:test --tests PlaintextAdminIntegrationTest.testConsumerGroupsDeprecatedConsumerGroupState --tests PlaintextAdminIntegrationTest.testConsumerGroups --tests PlaintextAdminIntegrationTest.testConsumerGroupWithMemberRemoval --rerun --fail-fast; do (( I=$I+1 )); echo "Completed run: $I"; sleep 1; done ``` -- 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]
