-
Notifications
You must be signed in to change notification settings - Fork 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
feat: Feature/kafkaoptions kwargs #4902
base: master
Are you sure you want to change the base?
feat: Feature/kafkaoptions kwargs #4902
Conversation
Signed-off-by: zerafachris PERSONAL <[email protected]>
Signed-off-by: zerafachris PERSONAL <[email protected]>
Signed-off-by: zerafachris PERSONAL <[email protected]>
Signed-off-by: zerafachris PERSONAL <[email protected]>
Signed-off-by: zerafachris PERSONAL <[email protected]>
With commit "1ef5d0afb35e3dd2f9f8aaee6da0a28f88982a82" I was able to add the proto definition for kafka_settings as a |
Signed-off-by: zerafachris PERSONAL <[email protected]>
I'll try to pull this down on Friday, sorry traveling for work. |
Hey @zerafachris mind taking a look? did you still need this? I was planning on cutting a release for it and wanted to make sure we get this in if needed. |
Signed-off-by: zerafachris PERSONAL <[email protected]>
Hi @franciscojavierarceo, I resolved the conflicts. Ideally, yes, I would be keen to use this. Unfortunately, I am still unable to fix the unit tests. The errors are:
|
I'll take a look at this today 👍 |
What this PR does / why we need it:
This is an attempt to introduce additional kafka settings to the Kafka stream as raised by #4894
Which issue(s) this PR fixes:
Not applicable.
Misc
Unfortunately, this is a partial solution and am looking for support on this.
I have made most of the necessary changes but unit tests are failing due to:
Any support is greatly appreciated