Skip to content

[client] Unify cluster inspection in DescribeCluster - #4901

Merged
wuchong merged 2 commits into
apache:mainfrom
swuferhong:merge-describe-cluster
Oct 27, 2026
Merged

wuchong merged 2 commits into
apache:mainfrom
swuferhong:merge-describe-cluster

Conversation

@swuferhong

Copy link
Copy Markdown
Contributor

Purpose

Linked issue: close #4900

Return cluster topology and tablet server replica loads through Admin.describeCluster(). Remove DescribeTabletServers and use API key 1075 for DESCRIBE_CLUSTER.

Keep getServerNodes on the Metadata discovery path and enforce cluster DESCRIBE authorization for inspection. Document coordinator routing, request volume guidance, and possible future tablet server routing.

Update protocol bindings and tests for authorization, missing loads, offline servers, and coordinator retries.

Brief change log

Tests

API and Format

Documentation

@swuferhong
swuferhong force-pushed the merge-describe-cluster branch from b258dfe to 9f41cd2 Compare October 27, 2026 02:05
swuferhong and others added 2 commits October 27, 2026 10:40
Unify cluster nodes and tablet server replica loads in Admin.describeCluster,
using RPC API key 1075. Keep getServerNodes on the Metadata discovery path
and enforce cluster DESCRIBE authorization for cluster inspection.

Document coordinator routing and request volume guidance, and update the
protocol bindings and coverage for authorization, missing loads and retries.
Classify NotCoordinatorLeaderException as InvalidMetadataException so
retryable calls refresh cluster metadata before retrying after a leader
change. Cover successful recovery and the one-retry limit.

Co-Authored-By: Codex <noreply@openai.com>
AI-Model: gpt-6
@swuferhong
swuferhong force-pushed the merge-describe-cluster branch from 9f41cd2 to a4faeff Compare October 27, 2026 02:41

@wuchong wuchong left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

LGTM.

@wuchong
wuchong merged commit e79e5bf into apache:main Oct 27, 2026
31 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.

[client] Unify cluster inspection through Admin.describeCluster()

2 participants