Skip to content
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.

Sign up for GitHub

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

Jump to bottom

release-20.2: sql: emit more tracing events from the stats cache #54190

Merged
merged 2 commits into from
Sep 16, 2020
Merged

release-20.2: sql: emit more tracing events from the stats cache #54190

merged 2 commits into from
Sep 16, 2020

Commits on Sep 10, 2020

  1. sql: emit more tracing events from the stats cache 

    The stats cache has various "slow" paths (where we need to query the
system table). These are currently only logged if verbosity is high.

This change switches to `VEvent` in most cases, so that these are
visible during tracing (including in statement diagnostics bundles).
This will allow us to diagnose slow planning times, e.g. due to the
stats cache getting full.

Release justification: low-risk change to existing functionality, high
potential benefit for debugging issues.

Release note: None
    @RaduBerinde
    RaduBerinde committed Sep 10, 2020
    Configuration menu
    Copy the full SHA
    8ce789a View commit details
    Browse the repository at this point in the history

  2. opt: add more tracing events around planning 

    Add a few more tracing events around the use of the optimizer.

Release note: None
    @RaduBerinde
    RaduBerinde committed Sep 10, 2020
    Configuration menu
    Copy the full SHA
    b79c514 View commit details
    Browse the repository at this point in the history