-
-
Notifications
You must be signed in to change notification settings - Fork 2.4k
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
Change message from warning to error when extension could not be loaded #8260
Change message from warning to error when extension could not be loaded #8260
Conversation
… print error instead of warning. Also, print clearer message.
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.
I think it's okay.
return Status(1, "Duplicate registry item: " + item.first); | ||
return Status(1, | ||
"Registry item " + item.first + | ||
" conflicts with another item of the same name."); |
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.
Why this change?
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.
Made the message a bit clearer. We had a customer hitting issues with duplicate table names, so we decided to look into this area and see if we could do something.
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.
Okay? I think "Duplicate" is pretty clear, and easy to grep for in logs. But I don't think I've hit it, and don't actually have an issue with the rewording.
Fixes #8254
When an extension cannot be loaded due to a duplicate item/extension name, print error instead of warning. Also, print clearer message for duplicate item.