-
Notifications
You must be signed in to change notification settings - Fork 3.1k
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
KIP 951 - Phase 1 #4613
KIP 951 - Phase 1 #4613
Conversation
adc98bf
to
bf47c62
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.
LGTM, just so minor changes. Please rebase
bf47c62
to
f02dc78
Compare
f02dc78
to
570913c
Compare
Rebased on top of #4595 |
9739010
to
25ab3f9
Compare
- Style fix - RPC version update - Fix uninitialized value and topic name from rkt - Revert newline changes - Remove duplicate declaration - Rename rd_kafka_buf_write_tags to rd_kafka_buf_write_tags_empty before implementing the define that allows to write tags - Upgrade fetch mock handler to v15 - devel assertion and logging improvements
25ab3f9
to
7b1ffd1
Compare
7b1ffd1
to
82a887b
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.
Some changes that will be needed for next commits
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.
Last few changes
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
Taken common structs from https://github.com/confluentinc/librdkafka/pull/4609/files.