ui: drop index with space#99690
Conversation
|
This change is |
maryliag
left a comment
There was a problem hiding this comment.
Done
Reviewable status:
complete! 0 of 0 LGTMs obtained
THardy98
left a comment
There was a problem hiding this comment.
Reviewed 3 of 3 files at r1, all commit messages.
Reviewable status:
Previously, if the index had a space on its name, it would fail to drop. This commit adds quotes so it can be executed. Fixes cockroachdb#97988 Release note (bug fix): Index recommendation to DROP an index that have a space on its name can now be properly executed.
|
bors r+ |
|
Build succeeded: |
|
Encountered an error creating backports. Some common things that can go wrong:
You might need to create your backport manually using the backport tool. error creating merge commit from e439695 to blathers/backport-release-22.2-99690: POST https://api.github.com/repos/cockroachdb/cockroach/merges: 409 Merge conflict [] you may need to manually resolve merge conflicts with the backport tool. Backport to branch 22.2.x failed. See errors above. 🦉 Hoot! I am a Blathers, a bot for CockroachDB. My owner is dev-inf. |
Previously, if the index had a space on its name,
it would fail to drop.
This commit adds quotes so it can be executed.
Fixes #97988
Schema Insights
https://www.loom.com/share/04363b7f83484b5da19c760eb8d0de21
Table Details page
https://www.loom.com/share/1519b897a14440ddb066fb2ab03feb2d
Release note (bug fix): Index recommendation to DROP an index that have a space on its name can now be properly executed.