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
release-23.1: ui: fix app = empty string filter on stmts page #108985
release-23.1: ui: fix app = empty string filter on stmts page #108985
Conversation
Thanks for opening a backport. Please check the backport criteria before merging:
If some of the basic criteria cannot be satisfied, ensure that the exceptional criteria are satisfied within.
Add a brief release justification to the body of your PR to justify this backport. Some other things to consider:
|
e819352
to
4b984d5
Compare
The filter on app name = empty string was not working on the stmts page. This was due to the fact that we use (unset) as the option in the filter to represent selecting the empty string app name. However when filtering statements, the empty string app name on the stmt was not changed accordingly. this commit fixes this and also adds testing for the unset case. This commit also adds sql api testing functions to mock stmt data, which was introduced in cockroachdb#101188. Epic: none Fixes: cockroachdb#107748 Release note (bug fix): Filter on stmts page works for app name = empty string (represented as 'unset').
4b984d5
to
5e0e699
Compare
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.
Your PR description is missing info such as which PR is being backported
Reviewed all commit messages.
Reviewable status: complete! 0 of 0 LGTMs obtained
Done. |
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.
Reviewable status: complete! 1 of 0 LGTMs obtained
Backport 1/1 commits from #107750
cc @cockroachdb/release
The filter on app name = empty string was not working on the stmts page. This was due to the fact that we use (unset) as the option in the filter to represent selecting the empty string app name. However when filtering statements, the empty string app name on the stmt was not changed accordingly. this commit fixes this and also adds testing for the unset case.
Epic: none
Fixes: #107748
Release note (bug fix): Filter on stmts page works for app name = empty string (represented as 'unset').
Release justification: bug fix