-
Notifications
You must be signed in to change notification settings - Fork 3.8k
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
ui: remove option 10/30 min from SQL Activity page #83229
Conversation
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.
Lgtm
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.
Reviewed 1 of 16 files at r1, 1 of 6 files at r2, all commit messages.
Reviewable status: complete! 1 of 0 LGTMs obtained (waiting on @kevin-v-ngo)
5fb02f5
to
60606a4
Compare
Previously we had the options for 10 and 30min on SQL Activity pages, which created some confusion, since we would always show the last 1h info. This commit remove those 2 options. If the user select any of those options on the Metrics page, it will get updated to 1h on the SQL Activity pages. Fixes cockroachdb#82914 Release note (ui change): Removal of the 10 and 30min options on the SQL Activity page.
60606a4
to
e4d7f25
Compare
bors r+ |
Build succeeded: |
The check for valid options with the removal of some options on cockroachdb#83229 didn't took the custom values into consideration. This commit add the option back, allowing the alignment on custom values. Release note (bug fix): Custom time period selection is now aligning between Metrics and SQL Activity page.
83619: opt: constrain expression indexes with IS NULL expressions r=mgartner a=mgartner The optimizer can generate constrained scans over indexes on computed columns when columns referenced in the computed column expression are held constant. Consider this example: CREATE TABLE t (a INT, v INT AS (a + 1) STORED, INDEX v_idx (v)) SELECT * FROM t WHERE a = 1 A constrained scan can be generated over `v_idx` because `v` depends on `a` and the query filter holds `a` constant. This commit lifts a restriction that prevented this optimization when columns referenced in the computed column expression were held constant to the `NULL` value. As far as I can tell, this restriction is not necessary. In fact, `@rytaft` had questioned its purpose originally, but the question was never answered: #43450 (review) By lifting this restriction, the optimizer can explore constrained scans over both indexed computed columns with `IS NULL` expressions and expression indexes with `IS NULL` expressions. Fixes #83390 Release note (performance improvement): The optimizer now explores more efficient query plans when index computed columns and expressions have `IS NULL` expressions. 84084: bazel: new versions of prebuilt `c-deps` r=srosenberg a=rickystewart Rebuild these archives to pull in `52a3a0aa8a707f9bb03802186da0c60b715ed9ce` (change to `jemalloc` to build without `MADV_FREE`). Release note: None 84088: ui: fix alignment on custom scale r=maryliag a=maryliag The check for valid options with the removal of some options on #83229 didn't took the custom values into consideration. This commit add the option back, allowing the alignment on custom values. Release note (bug fix): Custom time period selection is now aligning between Metrics and SQL Activity page. 84155: sql/schemachanger/scbuild: minor cleanup r=ajwerner a=ajwerner Improves the error handling a tad to make runtime errors and assertion failure. Fixes a typo. Release note: None Co-authored-by: Marcus Gartner <[email protected]> Co-authored-by: Ricky Stewart <[email protected]> Co-authored-by: Marylia Gutierrez <[email protected]> Co-authored-by: Andrew Werner <[email protected]>
The check for valid options with the removal of some options on #83229 didn't took the custom values into consideration. This commit add the option back, allowing the alignment on custom values. Release note (bug fix): Custom time period selection is now aligning between Metrics and SQL Activity page.
Note to reviewers: only 2nd commit is relevant to this PR
Previously we had the options for 10 and 30min on
SQL Activity pages, which created some confusion, since
we would always show the last 1h info.
This commit remove those 2 options.
If the user select any of those options on the Metrics
page, it will get updated to 1h on the SQL Activity
pages.
https://www.loom.com/share/226e54322df6456aa2039b5c54f72eb1
Fixes #82914
Release note (ui change): Removal of the 10 and 30min options
on the SQL Activity page.