cherrypick: fix lower case db name#25148
Conversation
Qodo reviews are paused for this user.Troubleshooting steps vary by plan Learn more → On a Teams plan? Using GitHub Enterprise Server, GitLab Self-Managed, or Bitbucket Data Center? |
|
@mergify refresh |
✅ Pull request refreshed |
XuPeng-SH
left a comment
There was a problem hiding this comment.
Re-checked the latest head. The previous blockers are fixed: protected_databases parsing no longer substitutes empty elements with the current database, and lower_case_table_names=2 is now treated consistently with the engine’s case-insensitive identifier semantics. The added unit/BVT coverage also pins those boundary cases, so I do not see a remaining correctness issue that should keep this blocked.
What type of PR is this?
Which issue(s) this PR fixes:
issue ##25079
What this PR does / why we need it:
修改 protected database lower case db name 问题