-
Notifications
You must be signed in to change notification settings - Fork 180
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
ruby-kafka is no longer maintained and superseded by librdkafka via rdkafka-ruby bindings #500
Comments
Yes, that's been a concern for us recently. About waterdrop I've just found out it now. |
Your implementation will cause Ruby VM crashes for sure in one place: you cannot destroy rdkafka object on timeout during close if there are ANY outgoing messages. Both this and the events queue needs to be purged: https://github.com/karafka/waterdrop/blob/master/lib/waterdrop/producer.rb#L186 otherwise once in a while there will be dangling data that librdkafka does not tolerate and causes VM failures. Anyhow keep me posted and happy to help :) |
Hey @mensfeld, to clarify, when you mentioned "The downside is jruby - which is something that will have to be figured out.", do you mean @ashie - I might quickly put together a PoC for |
That. It does NOT support jruby as it uses ffi and C bindings. |
Describe the bug
I just wanted to raise to your attention that based on the official statement from the ruby-kafka repo:
ruby-kafka has some critical errors that can lead to data loss. Even zendesk already switched to it with their other operations.
While I cannot replace it fully (rdkafka and waterdrop maintainer here) in this plugin because of my limited knowledge, I am happy to help you guys out :)
The downside is jruby - which is something that will have to be figured out.
To Reproduce
To reproduce or rather see, check the
.gemspec
Expected behavior
I would expect this library to use one of the maintained and recommended Kafka gem.
Your Environment
Your Configuration
Applies to any config.
Your Error Log
Additional context
Please close it if you feel this is not a bug and it should be raised as a discussion or something else. From my perspective it is a bug as it poses risk of data loss for the end users.
The text was updated successfully, but these errors were encountered: