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

Upgrade librdkafka to 1.6.1 and prepare release 0.11.7 #1228

Merged
merged 5 commits into from
Sep 28, 2021
Merged

Conversation

mfelsche
Copy link
Member

@mfelsche mfelsche commented Sep 28, 2021

Pull request

Description

Upgrade the dependency on librdkafka to 1.6.1

Related

Checklist

  • The RFC, if required, has been submitted and approved
  • Any user-facing impact of the changes is reflected in docs.tremor.rs
  • The code is tested
  • Use of unsafe code is reasoned about in a comment
  • Update CHANGELOG.md appropriately, recording any changes, bug fixes, or other observable changes in behavior
  • The performance impact of the change is measured (see below)

Matthias Wahl added 2 commits September 28, 2021 14:54
This mitigates confluentinc/librdkafka#2933 which is fixed in 1.5.2.
This issue was leading to hanging consumers when they tried to commit during a rebalance operation.

Signed-off-by: Matthias Wahl <mwahl@wayfair.com>
Signed-off-by: Matthias Wahl <mwahl@wayfair.com>
@mfelsche mfelsche changed the title Rdkafka upgrade Upgrade librdkafka to 1.5.2 Sep 28, 2021
@mfelsche mfelsche marked this pull request as ready for review September 28, 2021 12:58
@mfelsche mfelsche marked this pull request as draft September 28, 2021 13:26
@mfelsche mfelsche changed the title Upgrade librdkafka to 1.5.2 Upgrade librdkafka to 1.5.2 and prepare release 0.11.7 Sep 28, 2021
@mfelsche mfelsche marked this pull request as ready for review September 28, 2021 13:30
&self,
) -> &mut stream_consumer::MessageStream<'static, LoggingConsumerContext, SmolRuntime> {
&mut self,
) -> &mut stream_consumer::MessageStream<'_, LoggingConsumerContext, SmolRuntime> {
Copy link
Member

Choose a reason for hiding this comment

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

😭

Copy link
Member Author

Choose a reason for hiding this comment

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

pls don't cry. This change is for the better!

Copy link
Member

Choose a reason for hiding this comment

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

I know, but I'll be crying and sobbing in the corner until we finally can get rid of it, each transmute is a tear for me

Copy link
Member Author

Choose a reason for hiding this comment

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

Each unsafe transmute kills a little kitten when executed.

Copy link
Member

Choose a reason for hiding this comment

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

😭

Copy link
Member

@Licenser Licenser left a comment

Choose a reason for hiding this comment

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

🚀 🚢 it

Matthias Wahl added 3 commits September 28, 2021 16:17
Signed-off-by: Matthias Wahl <mwahl@wayfair.com>
Signed-off-by: Matthias Wahl <mwahl@wayfair.com>
Signed-off-by: Matthias Wahl <mwahl@wayfair.com>
@mfelsche mfelsche merged commit 7864749 into 0.11 Sep 28, 2021
@mfelsche mfelsche deleted the rdkafka-upgrade branch September 28, 2021 14:52
@mfelsche mfelsche changed the title Upgrade librdkafka to 1.5.2 and prepare release 0.11.7 Upgrade librdkafka to 1.6.1 and prepare release 0.11.7 Sep 29, 2021
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.

2 participants