[GitHub] [kafka] skaundinya15 commented on pull request #8421: KAFKA-9800: [KIP-580] Admin Client Exponential Backoff Implementation

[GitBox <gi...@apache.org>]

skaundinya15 commented on pull request #8421:
URL: https://github.com/apache/kafka/pull/8421#issuecomment-636725117


   @ijuma @abbccdda Initially the idea was to tackle one client at a time while making the patches for this KIP. With respect to creating an overarching module that can be used by all clients, the `AdminClient` might be unique in that it has different `Calls` it has to keep track of and schedule, whereas the backoff logic for the `Consumer` and `Producer` might look different. I think it's possible to be able to have some shared retry backoff logic for both the `Consumer` and `Producer`, but the `AdminClient`'s logic might have to be distinct from that. @d8tltanc, if you're going to follow up on this patch, consider looking at the client code for all the clients and see if the `AdminClient`'s logic could neatly fit in with the `Consumer` and `Producer`.


----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

For queries about this service, please contact Infrastructure at:
users@infra.apache.org