StarRocks / starrocks

The world's fastest open query engine for sub-second analytics both on and off the data lakehouse. With the flexibility to support nearly any scenario, StarRocks provides best-in-class performance for multi-dimensional analytics, real-time analytics, and ad-hoc queries. A Linux Foundation project.
https://starrocks.io
Apache License 2.0
9.18k stars 1.82k forks source link

[Enhancement] Optimize NodeSelector#seqChooseBackendOrComputeId() #52974

Open lmagic233 opened 1 week ago

lmagic233 commented 1 week ago

Why I'm doing:

Shared-data clusters do not necessarily (or should not) have BE nodes, thus invoking NodeSelector#seqChooseBackendOrComputeId() method will produce failed to find any backend logs, which are somewhat confusing.

What I'm doing:

Refactor that method to check for CN nodes under shared-data mode first, then check for BE under both modes.

What type of PR is this:

Does this PR entail a change in behavior?

If yes, please specify the type of change:

Checklist:

Bugfix cherry-pick branch check: