New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Fix ensure single-topic consumer can be closed #9849
Fix ensure single-topic consumer can be closed #9849
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Nice catch, could you please help add a unit test to cover this case to avoid regression?
Of course, I have added the unit test, please take a look @codelipenghui |
7c75543
to
8314c2e
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Nice catch
All checks have passed. Can this PR be merged? @codelipenghui @315157973 |
It is found that there is a logic problem in creating a consumer, which may cause the consumer to be unable to close (cherry picked from commit 1cd8658)
It is found that there is a logic problem in creating a consumer, which may cause the consumer to be unable to close