vladimirivic commented on issue #953: ZOOKEEPER-3398 Learner.connectToLeader() may take too long to time-out URL: https://github.com/apache/zookeeper/pull/953#issuecomment-494673203 @lvfangmin @eolivelli @anmolnar - what I am thinking is that it may be worth going through the code and see if this config could be generalized to be used exclusively for socket connection timeout wherever else we have similar logic. In that way we keep initLimit and syncLimit tied to quorum semantics where as this could be called socketTimeout (or some better name) which is somewhat on a lower level and belongs to the networking layer of Zookeeper. Not sure if this is a good idea, just had this thought. I would need to do some homework on my end but let me know what you think first.
---------------------------------------------------------------- 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: us...@infra.apache.org With regards, Apache Git Services
