airbnb / synapse

A transparent service discovery framework for connecting an SOA
MIT License
2.07k stars 251 forks source link

Remove on_connected to avoid reconnect storm #294

Closed anson627 closed 5 years ago

anson627 commented 5 years ago

We discovered zk client bug that could potentially cause watcher refresh storm and zk overload, when zk server restart

while we are waiting for the fix to get reviewed and well tested: https://github.com/airbnb/synapse/pull/292

For now remove on_connected callback to be safe

@Jason-Jian @allenlsy @Ramyak