zentol commented on a change in pull request #9759:
[FLINK-14169][history-server] Cleanup expired jobs from history server
URL: https://github.com/apache/flink/pull/9759#discussion_r336478340
##########
File path:
flink-core/src/main/java/org/apache/flink/configuration/HistoryServerOptions.java
##########
@@ -89,6 +89,16 @@
.withDescription("Enable HTTPs access to the
HistoryServer web frontend. This is applicable only when the" +
" global SSL flag security.ssl.enabled is set
to true.");
+ /**
+ * Specifies whether the HistoryServer should process job archive
deletion. If this option is
+ * enabled then deleted job archives are also deleted from
HistoryServer.
+ */
+ public static final ConfigOption<Boolean>
HISTORY_SERVER_CLEANUP_EXPIRED_JOBS =
+ key("historyserver.web.clean-expired-jobs")
Review comment:
the web prefix doesn't make sense here; web options are exclusively for the
WebUI portion of the HS. `historyserver.archive` is a more appropriate config
key.
----------------------------------------------------------------
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:
[email protected]
With regards,
Apache Git Services
