search

oleksiyk / kafka

Apache Kafka 0.9 client for Node
MIT License
297 stars 85 forks source link

ReconnectFix #249

Closed larryckey closed 5 years ago

larryckey commented 5 years ago

step to reproduce problem in previous version:

start your service without Kafka and Zookeeper bring up Kafka and Zookeeper and service won't connect after embedding this fix, step 2 will connect

oleksiyk commented 5 years ago

This code has been intentionally removed in #240. The reason has been described in #239

larryckey commented 5 years ago

Hi, you completely remove all connection retries? why not just limit it to X times? the problem is: if my service (service that use 'no-kafka' package) is up before 'Kafka' is up, it will never connect