airbnb / synapse

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

Set explicit zk session timeout #327

Closed gmcatsf closed 4 years ago

gmcatsf commented 4 years ago

Summary

Synapse does not specify explicit value for session timeout. By default ruby client uses RECEIVE_TIMEOUT=10000ms which leads to 20-sec session in zk server.

https://github.com/zk-ruby/zookeeper/blob/master/ext/c_zookeeper.rb#L20

The PR sets synapse session timeout to be 36-sec.

Test

Reviewers

@panchr @austin-zhu @Jason-Jian

gmcatsf commented 4 years ago

updated version to 0.18.3