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 to Confluent.Kafka 1.5.2 #185

Closed
amotl opened this issue Oct 15, 2020 · 8 comments
Closed

Upgrade to Confluent.Kafka 1.5.2 #185

amotl opened this issue Oct 15, 2020 · 8 comments
Labels
enhancement New feature or request

Comments

@amotl
Copy link

amotl commented Oct 15, 2020

Dear @TsuyoshiUshio, @fbeltrao and @ryancrawcour,

as I see you have upgraded to Confluent.Kafka 1.4.3 within #151 the other day, I would like to humbly point out confluentinc/librdkafka#3109 to you.

Without using the Azure Kafka bindings yet, we and others tripped into different issues when connecting to Event Hubs with librdkafka <1.5.0 through its Python binding confluent-kafka-python when running within a Kubernetes environment on Azure.

Now, as I believe the same things might also apply to the Azure Kafka bindings, I wanted to give you a heads up on this.

Without any offense to you or the networking infrastructure within Azure, citing others:

TL;DR: Azure has a nasty artificial limitation that results in being unable to use long-lived TCP connections that have >= 4 minutes of radio silence at any given point.

After some issues revolving around that had been popping up on the Confluent's issue trackers, @edenhill was able to add some mitigations to librdkafka just recently.

Apart from upgrading to the most recent version of librdkafka, I would like to emphasize that it is probably also crucial to apply the recommended configuration settings for Kafka on Azure as outlined within confluentinc/librdkafka#3109 (comment).

After applying these measures to our systems, things have improved tremendously.

With kind regards,
Andreas.

@TsuyoshiUshio
Copy link
Contributor

Thank you for sharing it, @amotl

Absolutely, I'd like to upgrade it once it comes. I'd like to upgrade the code to adopt the recommended configuration for Azure.
I appreciate your contribution. I also might ping him to heads up the intermediate certificate issue.

@amotl
Copy link
Author

amotl commented Oct 20, 2020

Dear Tsuyoshi,

librdkafka 1.5.2 GA has just been released, see [1]. For making progress on that, we will probably have to wait for the bump propagating to confluent-kafka-dotnet and being available on NuGet [2].

With kind regards,
Andreas.

[1] https://github.com/edenhill/librdkafka/releases/tag/v1.5.2
[2] https://www.nuget.org/packages/Confluent.Kafka/

@TsuyoshiUshio
Copy link
Contributor

Thanks. It's a good new. Once it is released. I'll release this library with it.

@amotl
Copy link
Author

amotl commented Oct 20, 2020

Once it is released, I'll release this library with it.

Thanks. @mhowlett is on it, see confluentinc/confluent-kafka-dotnet#1432.

@amotl
Copy link
Author

amotl commented Oct 21, 2020

Confluent.Kafka 1.5.2 has been released, see https://www.nuget.org/packages/Confluent.Kafka/. Thanks a bunch, @mhowlett and @edenhill!

@TsuyoshiUshio
Copy link
Contributor

It's a good news I'll include it

@TsuyoshiUshio
Copy link
Contributor

Sounds good? #186

@amotl
Copy link
Author

amotl commented Oct 27, 2020

Thanks a again for considering to upgrade to 1.5.2! Closing this now.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

No branches or pull requests

2 participants