Skip to content

[fix] correct rebalance-type FE config key in managing-compute-cluster doc#3693

Merged
morningman merged 1 commit into
apache:masterfrom
boluor:fix-managing-compute-cluster-config-key
May 19, 2026
Merged

[fix] correct rebalance-type FE config key in managing-compute-cluster doc#3693
morningman merged 1 commit into
apache:masterfrom
boluor:fix-managing-compute-cluster-config-key

Conversation

@boluor
Copy link
Copy Markdown
Contributor

@boluor boluor commented May 19, 2026

Summary

The managing-compute-cluster doc referenced an FE config cloud_default_rebalance_type in two places — the fe.conf example and the ADMIN SHOW FRONTEND CONFIG example — but no such config exists in Doris.

The real config is cloud_warm_up_for_rebalance_type (default async_warmup), verified against the apache/doris FE source: it is defined in Config.java and used in BalanceTypeEnum.java, ComputeGroup.java, CloudTabletRebalancer.java, and the cloud regression tests. The ADMIN SET FRONTEND CONFIG example in the same doc already used the correct key — only the other two references were wrong.

Corrected cloud_default_rebalance_typecloud_warm_up_for_rebalance_type across all affected versions (EN next/4.x, ZH next/4.x; the feature does not exist in 3.x).

Test plan

  • Verified against apache/doris Config.java: cloud_warm_up_for_rebalance_type exists, cloud_default_rebalance_type does not (zero matches repo-wide)
  • Confirmed the ADMIN SET example and the cloud regression tests use cloud_warm_up_for_rebalance_type

…r doc

The doc referenced `cloud_default_rebalance_type` in the fe.conf example
and the ADMIN SHOW FRONTEND CONFIG example, but no such FE config exists.
The real config is `cloud_warm_up_for_rebalance_type` (verified in the
apache/doris FE source: Config.java, BalanceTypeEnum.java, ComputeGroup.java).
Corrected the key across all affected EN and Chinese versions.

Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
@morningman morningman merged commit 25672b2 into apache:master May 19, 2026
3 checks passed
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.

2 participants