Skip to content

ui: extend search logic on insights page#107716

Merged
craig[bot] merged 1 commit intocockroachdb:masterfrom
koorosh:ui-fix-search-on-insights-page
Jul 27, 2023
Merged

ui: extend search logic on insights page#107716
craig[bot] merged 1 commit intocockroachdb:masterfrom
koorosh:ui-fix-search-on-insights-page

Conversation

@koorosh
Copy link
Copy Markdown
Contributor

@koorosh koorosh commented Jul 27, 2023

This change extends the number of fields where
search is applied (instead of single transaction/
statement execution ID field).
It makes possible to search for any available ID
in Txn or statement insight.

Release note (ui change): search is performed on all ID fields of transaction and statement insights.

Resolves: #107253

Demo:

Screen.Recording.2023-07-27.at.22.14.25.mov

This change extends the number of fields where
search is applied (instead of single transaction/
statement execution ID field).
It makes possible to search for any available ID
in Txn or statement insight.

Release note (ui change): search is performed on all
ID fields of transaction and statement insights.
@koorosh koorosh requested review from a team, THardy98, gtr, j82w, maryliag, xinhaoz and zachlite July 27, 2023 14:38
@blathers-crl
Copy link
Copy Markdown

blathers-crl bot commented Jul 27, 2023

Thank you for contributing to CockroachDB. Please ensure you have followed the guidelines for creating a PR.

My owl senses detect your PR is good for review. Please keep an eye out for any test failures in CI.

🦉 Hoot! I am a Blathers, a bot for CockroachDB. My owner is dev-inf.

@blathers-crl blathers-crl bot added the O-community Originated from the community label Jul 27, 2023
@cockroach-teamcity
Copy link
Copy Markdown
Member

This change is Reviewable

Copy link
Copy Markdown
Contributor

@maryliag maryliag left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

can you add a demo of this working?

Reviewable status: :shipit: complete! 0 of 0 LGTMs obtained (waiting on @gtr, @j82w, @THardy98, @xinhaoz, and @zachlite)

@koorosh
Copy link
Copy Markdown
Contributor Author

koorosh commented Jul 27, 2023

can you add a demo of this working?

Reviewable status: :shipit: complete! 0 of 0 LGTMs obtained (waiting on @gtr, @j82w, @THardy98, @xinhaoz, and @zachlite)

Done!

Copy link
Copy Markdown
Contributor

@maryliag maryliag left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

:lgtm:

Reviewable status: :shipit: complete! 1 of 0 LGTMs obtained (waiting on @gtr, @j82w, @THardy98, @xinhaoz, and @zachlite)

@koorosh
Copy link
Copy Markdown
Contributor Author

koorosh commented Jul 27, 2023

bors r+

@koorosh koorosh added the backport-23.1.x PAST MAINTENANCE SUPPORT: 23.1 patch releases via ER request only label Jul 27, 2023
@craig
Copy link
Copy Markdown
Contributor

craig bot commented Jul 27, 2023

Build succeeded:

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

backport-23.1.x PAST MAINTENANCE SUPPORT: 23.1 patch releases via ER request only O-community Originated from the community

Projects

None yet

Development

Successfully merging this pull request may close these issues.

insights: search does not include fingerprint

3 participants