-
-
Notifications
You must be signed in to change notification settings - Fork 61
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
Improve kafka adapter #411
Comments
/gfi ts |
Hey @KhudaDad414 , I would like to work on this issue |
@shubham-y sure. Just let me know if you have any questions 🙂 |
I see there is a similar draft PR #381 opened 2 months ago. So is there any discussion that I can refer to from that time? |
Not that I know of. this issue is also kinda related to #27 |
Hey @KhudaDad414 , I went through the current implementation of kafka adapter and kafka bindings document. For further doubts in the future, please let me know which platform is preferred for such doubts Slack or GitHub comments |
Hi @shubham-y,
Based on a recent discussion that I had with @fmvilas, We think it is better to ignore the |
Hey @shubham-y, Have you started working on this issue? 😄 @Ruchip16 can continue to work on it or give a hand if you need some help. |
Hey @KhudaDad414, I had started working on it but had to take a break due to unavoidable personal circumstances. Sorry, I did not communicate earlier. I have resumed working on it and it will great if @Ruchip16 can help me with this as I am struggling a bit due to limited docs/tutorials for Glee. |
hey I can initiate a PR for you and you can take it from there, if that sounds good? |
Sounds good. Thank you for that. |
Me and @shubham-y have agreed to work on it together 🙂 UPDATE (May 26) |
Hi, is this issue available to be taken? |
@Ruchip16 is working on it @sudoshreyansh 🙂 I wanted to take it up but she said she was working on it. |
/progress 10 I'm starting to work on this. |
any updates on this @oviecodes? |
@Souvikns I'm sorry I got so busy, please can you take this, I'll join you once I'm free. Thanks |
@KhudaDad414 will take a look |
/progress 10 |
@KhudaDad414 any updates on this? |
@oviecodes I am busy with spec v3 support in glee. might do it later. I don't have any progress in it. so feel free to pick it up. |
Alright man 👍... |
@fmvilas @Souvikns @KhudaDad414 could anyone please explain this issue further? |
@oviecodes to summarise, you have to look through the bindings here: https://github.com/asyncapi/bindings/tree/next-major-spec/kafka, parse them from the asyncapi document and apply them whenever necessary. we are using https://kafka.js.org/ in Glee, so all you have to do is parse the bindings and pass them to the |
Hey @KhudaDad414 can i start working on this again, I promise to complete it this time 😅? |
@Ruchip16 wanna work on this one together? 😃 |
yes yes suree 😊 |
that's great, we'll open a pull request to update our progress on this issue 🙂 |
/progress 20 looking into kafka bindings and kafka.js |
The Kafka Adapter has to be updated to use spec v3. I'll open an issues as appropriate @Souvikns @KhudaDad414 |
This issue has been automatically marked as stale because it has not had recent activity 😴 It will be closed in 120 days if no further activity occurs. To unstale this issue, add a comment with a detailed explanation. There can be many reasons why some specific issue has no activity. The most probable cause is lack of time, not lack of interest. AsyncAPI Initiative is a Linux Foundation project not owned by a single for-profit company. It is a community-driven initiative ruled under open governance model. Let us figure out together how to push this issue forward. Connect with us through one of many communication channels we established here. Thank you for your patience ❤️ |
still relevant. |
This issue has been automatically marked as stale because it has not had recent activity 😴 It will be closed in 120 days if no further activity occurs. To unstale this issue, add a comment with a detailed explanation. There can be many reasons why some specific issue has no activity. The most probable cause is lack of time, not lack of interest. AsyncAPI Initiative is a Linux Foundation project not owned by a single for-profit company. It is a community-driven initiative ruled under open governance model. Let us figure out together how to push this issue forward. Connect with us through one of many communication channels we established here. Thank you for your patience ❤️ |
Reason/Context
The current implementation of kafka adapter provides the bare minimum functionality and I think we can improve upon that by using bindings.
What are bindings
AsyncAPI is protocol agnostic, it means that it supports a wide range of protocols and doesn't "assume" the protocol when you are writing your AsyncAPI file.
but if it doesn't assume protocols, how are we supposed to write our file accustomed to an specific protocols?
Behold, the answer is
bindings
.If you look at kafka bindings you will see a wide range of options that can be used to customize every part of your
kafka
client.The Problem
Currently kafka adapter ignore lots of this options. for example. instead of using
clientId
from bindings forclinetId
it just sets it toglee-client
.The Solution
Kafka bindings should be examined and we should support as much features as the bindings support.
The text was updated successfully, but these errors were encountered: