-
Notifications
You must be signed in to change notification settings - Fork 552
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
Remove duplicate responses sent by offset_fetch request handler #8689
Remove duplicate responses sent by offset_fetch request handler #8689
Conversation
- This request handler returns a response that contains an array of topics and within each topic, another array of partitions. - Within each array of partitions, a duplicate entry was being created, since push_back was invoked right after emplace_back, twice for each iteration of the loop to intitialize all partitions.
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.
LGTM
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
/backport v22.3.x |
Failed to run cherry-pick command. I executed the below command:
|
@vshtokman probably a merge conflict. author (or someone) will need to do the backport manually. |
[v22.3.x] Backport of #8689 Remove duplicate responses sent by offset_fetch request handler
This request handler returns a response that contains an array of topics and within each topic, another array of partitions.
Within each array of partitions, a duplicate entry was being created, since push_back was invoked right after emplace_back, twice for each iteration of the loop to intitialize all partitions.
Backports Required
Release Notes
Bug Fixes