Skip to content

Conversation

@RocMarshal
Copy link
Contributor

What is the purpose of the change

[FLINK-38716][docs] Describe the impact of slot.request.max-interval on balanced tasks scheduling.

Brief change log

[FLINK-38716][docs] Describe the impact of slot.request.max-interval on balanced tasks scheduling.

Verifying this change

N.A

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

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

Documentation

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

@flinkbot
Copy link
Collaborator

flinkbot commented Nov 25, 2025

CI report:

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

<span class="label label-info">Note</span> During failover scenarios,
when resources are released and resource requests are processed,
the delayed updates in the resource view may lead to allocation results that are not optimally balanced.
In such cases, you can improve the situation by appropriately increasing the value of
Copy link
Contributor

@davidradl davidradl Nov 25, 2025

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

To make this clearer, I suggest:

  • being explicit as to what you mean by appropriately
  • being explicit as to what you mean by sub-optimal and what optimal would look like.
  • it would be helpful to describe why increasing slot.request.max-interval can help, and maybe what happens if it is too high.
  • Maybe give an example to illustrate the points you are making in the text.

@github-actions github-actions bot added the community-reviewed PR has been reviewed by the community. label Nov 25, 2025
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