nicoweidner commented on pull request #15675:
URL: https://github.com/apache/flink/pull/15675#issuecomment-892712080


   > @nicoweidner I'd suggest naming the config option 
`tolerate-suspended-connections` to make it clear that we do not tolerate 
Curator's LOST (expired Zk sessions). I think it is actually debatable whether 
we need another config option. I think we would want the best possible 
experience out of the box. Is there any situation where we would the current 
behavior?
   
   This is a better and more precise naming suggestion for the config option!
   
   I took the point about keeping current behavior as default from a 
conversation with @tillrohrmann. The reason might be fear of unforeseen 
consequences when getting this in at the last minute (so it does not have much 
exposure before the release) - I will get him to chime in on this topic.


-- 
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: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]


Reply via email to