nicoweidner commented on a change in pull request #16801:
URL: https://github.com/apache/flink/pull/16801#discussion_r688589525
##########
File path:
docs/layouts/shortcodes/generated/high_availability_configuration.html
##########
@@ -62,6 +62,12 @@
<td>Integer</td>
<td>Defines the session timeout for the ZooKeeper session in
ms.</td>
</tr>
+ <tr>
+
<td><h5>high-availability.zookeeper.client.tolerate-suspended-connections</h5></td>
+ <td style="word-wrap: break-word;">false</td>
+ <td>Boolean</td>
+ <td>Defines whether a suspended ZooKeeper connection will be
treated as an error that causes the leader information to be invalidated or
not. In case you set this option to <code
class="highlighter-rouge">true</code>, Flink will wait until a ZooKeeper
connection is marked as lost before it revokes the leadership of components.
This has the effect that Flink is more resilient against temporary connection
instabilities at the cost of running more likely into timing issues with
ZooKeeper.</td>
Review comment:
Hm... alright, I don't know much about ZK internals. I guess it will
depend on the various timeouts (in particular session expiration, connection
loss), so the warning should suffice.
--
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]
