Skip to content
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

bug: V5 MetadataRequest nullable topics array #1353

Merged
merged 1 commit into from
Apr 12, 2019
Merged

bug: V5 MetadataRequest nullable topics array #1353

merged 1 commit into from
Apr 12, 2019

Conversation

dnwe
Copy link
Collaborator

@dnwe dnwe commented Apr 10, 2019

Consulting the protocol guide the V5 request uses the same nullable
array for topics as the V1-V4 requests. Improve existing unittest to
properly exercise the encoding paths.

Fixes #1352

Consulting the protocol guide the V5 request uses the same nullable
array for topics as the V1-V4 requests. Improve existing unittest to
properly exercise the encoding paths.

Fixes #1352

Signed-off-by: Dominic Evans <dominic.evans@uk.ibm.com>
Copy link
Contributor

@varun06 varun06 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@bai
Copy link
Contributor

bai commented Apr 12, 2019

Thank you!

@bai bai merged commit c14ccca into IBM:master Apr 12, 2019
Mongey added a commit to Mongey/terraform-provider-kafka that referenced this pull request Apr 14, 2019
Mongey added a commit to Mongey/terraform-provider-kafka that referenced this pull request Apr 14, 2019
Mongey added a commit to Mongey/terraform-provider-kafka that referenced this pull request Apr 14, 2019
Mongey added a commit to Mongey/terraform-provider-kafka that referenced this pull request Apr 14, 2019
Mongey added a commit to Mongey/terraform-provider-kafka that referenced this pull request Apr 14, 2019
Master rather than v1.22.0 as we need fix for sarama bug
IBM/sarama#1353
Mongey added a commit to Mongey/terraform-provider-kafka that referenced this pull request Apr 14, 2019
Master rather than v1.22.0 as we need fix for sarama bug
IBM/sarama#1353
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

MetadataRequest V5 has wrong encoding for "all topics"
3 participants