Skip to content

chore(scheduler): remove the no-op disable-stage-plan-cache feature - #2836

Merged
milenkovicm merged 1 commit into
apache:mainfrom
NoahKusaba:chore/scheduler-remove-disable-stage-plan-cache
Nov 10, 2026
Merged

milenkovicm merged 1 commit into
apache:mainfrom
NoahKusaba:chore/scheduler-remove-disable-stage-plan-cache

Conversation

@NoahKusaba

Copy link
Copy Markdown
Contributor

Which issue does this PR close?

No issue filed; small cleanup.

Rationale for this change

#1266 added the disable-stage-plan-cache feature to turn off the scheduler's cache of encoded stage plans. #2038 (released in 55.0.0) removed that cache and now encodes each task's plan separately, but left the feature behind. Nothing reads it, yet both READMEs still document it.

What changes are included in this PR?

  • Remove the feature from ballista/scheduler/Cargo.toml.
  • Drop it from the feature tables in README.md and ballista/scheduler/README.md.
  • Note the removal in the 56.0.0 upgrade guide.

Are there any user-facing changes?

Behavior is unchanged. Anyone enabling the feature on their ballista-scheduler dependency must remove it from their features list. This should get the api-change label.

The scheduler stopped caching encoded stage plans in 55.0.0 (apache#2038) and
now encodes each task's plan separately, so nothing reads this feature.
Remove it, drop it from the two README feature tables, and note the
removal in the 56.0.0 upgrade guide.
@github-actions github-actions Bot added the documentation Improvements or additions to documentation label Nov 10, 2026

@milenkovicm milenkovicm left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

thanks @NoahKusaba

@milenkovicm
milenkovicm added this pull request to the merge queue Nov 10, 2026
Merged via the queue into apache:main with commit b0f4e94 Nov 10, 2026
26 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

documentation Improvements or additions to documentation

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants