Myracle commented on code in PR #27509:
URL: https://github.com/apache/flink/pull/27509#discussion_r2867159147
##########
docs/layouts/shortcodes/generated/web_configuration.html:
##########
@@ -26,6 +26,12 @@
<td>Integer</td>
<td>Number of checkpoints to remember for recent history.</td>
</tr>
+ <tr>
+ <td><h5>web.execution-graph.cache-ttl</h5></td>
+ <td style="word-wrap: break-word;">(none)</td>
+ <td>Duration</td>
+ <td>Time-to-live for cached ExecutionGraph. If not set, defaults
to the value of '<code class="highlighter-rouge">web.refresh-interval</code>'.
Setting this to 0 means the cache will always fetch fresh data, which is useful
for real-time state synchronization scenarios.</td>
Review Comment:
Thanks for the review! You are absolutely right. The web. prefix is reserved
for Web UI-facing configurations, and execution-graph is a Flink internal
concept that doesn't fit in this namespace. I've moved the config option to
RestOptions with the key rest.cache.execution-graph.timeout, following the same
naming convention as the existing rest.cache.checkpoint-statistics.timeout. The
documentation has been updated accordingly.
--
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]
