-
Notifications
You must be signed in to change notification settings - Fork 6.6k
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
fix storage policy prompt display error #51519
Conversation
src/Disks/StoragePolicy.cpp
Outdated
@@ -302,15 +302,15 @@ void StoragePolicy::checkCompatibleWith(const StoragePolicyPtr & new_storage_pol | |||
for (const auto & volume : getVolumes()) | |||
{ | |||
if (!new_volume_names.contains(volume->getName())) | |||
throw Exception(ErrorCodes::BAD_ARGUMENTS, "New storage policy {} shall contain volumes of old one", backQuote(name)); | |||
throw Exception(ErrorCodes::BAD_ARGUMENTS, "New storage policy {} shall contain volumes of old one", backQuote(new_storage_policy->getName())); |
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.
Let's print both names, new and old.
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.
good idea, I've got it
This is an automated comment for commit 5f523e8 with description of existing statuses. It's updated for the latest CI running
|
This comment was marked as abuse.
This comment was marked as abuse.
Wait while it is fixed, or try to figure it out on your own. |
OK,thanks for your reply |
@alexey-milovidov @evillique Hello,Please help me merge this PR,thanks. |
Changelog category (leave one):
Changelog entry (a user-readable short description of the changes that goes to CHANGELOG.md):
correcting the message of modify storage policy #51516
Documentation entry for user-facing changes