Skip to content

[FLINK-40478][runtime-web] Log effective HistoryServer configuration at startup - #29020

Open
argoyal2212 wants to merge 1 commit into
apache:masterfrom
argoyal2212:argoyal+01/history-server-startup-config-logging
Open

[FLINK-40478][runtime-web] Log effective HistoryServer configuration at startup#29020
argoyal2212 wants to merge 1 commit into
apache:masterfrom
argoyal2212:argoyal+01/history-server-startup-config-logging

Conversation

@argoyal2212

@argoyal2212 argoyal2212 commented Aug 26, 2026

Copy link
Copy Markdown
Contributor

What is the purpose of the change

Several HistoryServer configuration options change its runtime behavior in meaningful ways: which archive storage backend it uses (FILE vs ROCKSDB), the archive load mode (EAGER vs LAZY), and the retention policy (retained job/application counts, TTL, cleanup flags). None of these are logged today, they are just read into local variables and used silently.

This makes it hard to confirm from the logs what configuration actually took effect on a given HistoryServer instance. A misconfigured or overridden option can go unnoticed until something looks wrong in the UI, at which point the only way to check is to read the config file directly on the host or dig through source code.

Brief change log

  • Added a single log line in the HistoryServer constructor that reports the effective archive storage type, load mode, retained-jobs, retained-applications, retained-ttl, and clean-expired-jobs/applications settings.

Verifying this change

This change is a trivial rework / code cleanup without any test coverage.

This change is already covered by existing tests, such as HistoryServerTest and HistoryServerArchiveFetcherTest, which continue to pass unmodified since no runtime behavior changes.

Does this pull request potentially affect one of the following parts:

  • Dependencies (does it add or upgrade a dependency): no
  • The public API, i.e., is any changed class annotated with @Public(Evolving): no
  • The serializers: no
  • The runtime per-record code paths (performance sensitive): no
  • Anything that affects deployment or recovery: JobManager (and its components), Checkpointing, Kubernetes/Yarn, ZooKeeper: no
  • The S3 file system connector: no

Documentation

  • Does this pull request introduce a new feature? no
  • If yes, how is the feature documented? not applicable

Was generative AI tooling used to co-author this PR?
  • Yes (please specify the tool below)

Adds storage type, load mode, and retention settings to the startup log.
@flinkbot

flinkbot commented Aug 26, 2026

Copy link
Copy Markdown
Collaborator

CI report:

Bot commands The @flinkbot bot supports the following commands:
  • @flinkbot run azure re-run the last Azure build

@Jackeyzhe

Copy link
Copy Markdown
Contributor

Thanks for the patch. I checked this together with #29008 since the PR description says this is a follow-up to feedback there. Would it be better to fold this logging change back into #29008 instead of keeping it as a separate PR?

@argoyal2212 argoyal2212 changed the title [FLINK-XXXXX][runtime-web] Log effective HistoryServer configuration at startup [FLINK-40478][runtime-web] Log effective HistoryServer configuration at startup Aug 26, 2026
@argoyal2212

Copy link
Copy Markdown
Contributor Author

Thanks for the patch. I checked this together with #29008 since the PR description says this is a follow-up to feedback there. Would it be better to fold this logging change back into #29008 instead of keeping it as a separate PR?

Thanks for taking a look @Jackeyzhe. I did not mean to have this as a followup rather keep it separate from #29008. Filed a new jira and linked this PR.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants