Skip to content

[FLINK-39900] [forst] Add configuration option for setting the number of ForSt transfer threads#28373

Open
francis-a wants to merge 7 commits into
apache:masterfrom
francis-a:feature-FLINK-39900
Open

[FLINK-39900] [forst] Add configuration option for setting the number of ForSt transfer threads#28373
francis-a wants to merge 7 commits into
apache:masterfrom
francis-a:feature-FLINK-39900

Conversation

@francis-a

@francis-a francis-a commented Jun 10, 2026

Copy link
Copy Markdown

What is the purpose of the change

Currently the number of threads used by the ForStStateDataTransfer class is hardcoded to 4. This PR proposes a new state.backend.forst.checkpoint.transfer.thread.num configuration option that allows for providing a custom thread number value.

Brief change log

  • Added a new state.backend.forst.checkpoint.transfer.thread.num config option
  • Updated the ForStResourceContainer with a getter to return this option value
  • Updated ForStStateDataTransfer to provide the new config option value instead of the hardcoded default of 4
  • Removed the hardcoded default

Verifying this change

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

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: (yes / no / don't know)

Documentation

  • Does this pull request introduce a new feature? yes
  • If yes, how is the feature documented? Documentation has been added using the existing ConfigOption class

@flinkbot

flinkbot commented Jun 10, 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

francis-a and others added 3 commits June 10, 2026 16:02
…g/apache/flink/state/forst/snapshot/ForStIncrementalSnapshotStrategyTest.java

Co-authored-by: Purushottam Sinha <sinhapurushottam911@gmail.com>
@francis-a francis-a requested a review from spuru9 June 10, 2026 14:12
@github-actions github-actions Bot added the community-reviewed PR has been reviewed by the community. label Jun 10, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

community-reviewed PR has been reviewed by the community.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants