Skip to content

sql/stats: silence noisy autostats log message#133899

Merged
craig[bot] merged 1 commit intocockroachdb:masterfrom
michae2:silence_autostats
Oct 31, 2024
Merged

sql/stats: silence noisy autostats log message#133899
craig[bot] merged 1 commit intocockroachdb:masterfrom
michae2:silence_autostats

Conversation

@michae2
Copy link
Copy Markdown
Collaborator

@michae2 michae2 commented Oct 30, 2024

We currently only allow a single automatic stats collection at a time. If a stats collection is running for a long time, any queued automatic stats collections will log the following message once a minute:

automatically executing "CREATE STATISTICS __auto__ FROM [... "

This is expected, but we don't need to fill the logs with this message. Let's only log it if verbose logging is enabled.

Epic: None

Release note: None

We currently only allow a single automatic stats collection at a time.
If a stats collection is running for a long time, any queued automatic
stats collections will log the following message once a minute:

```
automatically executing "CREATE STATISTICS __auto__ FROM [... "
```

This is expected, but we don't need to fill the logs with this message.
Let's only log it if verbose logging is enabled.

Epic: None

Release note: None
@michae2 michae2 requested review from a team, dt and rytaft October 30, 2024 22:41
@blathers-crl
Copy link
Copy Markdown

blathers-crl bot commented Oct 30, 2024

It looks like your PR touches production code but doesn't add or edit any test code. Did you consider adding tests to your PR?

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

@cockroach-teamcity
Copy link
Copy Markdown
Member

This change is Reviewable

Copy link
Copy Markdown
Collaborator

@rytaft rytaft left a comment

Choose a reason for hiding this comment

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

:lgtm:

Reviewed 1 of 1 files at r1, all commit messages.
Reviewable status: :shipit: complete! 1 of 0 LGTMs obtained (waiting on @dt)

@michae2
Copy link
Copy Markdown
Collaborator Author

michae2 commented Oct 31, 2024

Thanks!

bors r=rytaft,dt

@craig
Copy link
Copy Markdown
Contributor

craig bot commented Oct 31, 2024

@craig craig bot merged commit 95a9d8d into cockroachdb:master Oct 31, 2024
@michae2 michae2 deleted the silence_autostats branch October 31, 2024 23:12
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants