-
Notifications
You must be signed in to change notification settings - Fork 7.1k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Add some columns to system.clusters #48548
Conversation
The edited test has failed. |
This is an automated comment for commit 451f02b with description of existing statuses. It's updated for the latest CI running
|
@alexey-milovidov, tests are OK now, would you like to review it? |
if (is_active.empty()) | ||
res_columns[i++]->insertDefault(); | ||
else | ||
res_columns[i++]->insert(is_active[replica_idx++]); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This part is difficult to understand: there is replica_index, and also replica_idx, and for some reason, it is incremented for every shard and replica, but only if there is is_active
...
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Everything else LGTM, merge at your convenience.
Sorry, missed this... LGTM. |
Changelog category (leave one):
Changelog entry (a user-readable short description of the changes that goes to CHANGELOG.md):
Added new columns with info about
Replicated
database replicas tosystem.clusters
:database_shard_name
,database_replica_name
,is_active
. Added an optionalFROM SHARD
clause toSYSTEM DROP DATABASE REPLICA
query.Documentation entry for user-facing changes