mimaison commented on PR #11565:
URL: https://github.com/apache/kafka/pull/11565#issuecomment-1119452581

   Sorry for the delay, but this finally made it back to the top of my review 
queue.
   
   I'm not sure we necessarily need to do something here. Starting Connect from 
scratch when not enough brokers are running must be a pretty rare scenario 
(maybe more common when developing). Connect has no idea when, or even if, 
enough brokers will join the cluster so it's hard to decide whether to keep 
retrying to just stop and let the environment (for example k8s) handle the 
restarts.
   
   If you definitively want Connect to do some retries, maybe we could retry 
for `default.api.timeout.ms`. I typically find setting a bound as a duration 
more intuitive than setting a number of retries. Also having separate retries 
settings for each topic may be a bit overkill :)


-- 
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.

To unsubscribe, e-mail: jira-unsubscr...@kafka.apache.org

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

Reply via email to