Skip to content

sql/stats: enable automatic collection of partial table stats#133988

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

sql/stats: enable automatic collection of partial table stats#133988
craig[bot] merged 1 commit intocockroachdb:masterfrom
michae2:enable_autopartialstats

Conversation

@michae2
Copy link
Copy Markdown
Collaborator

@michae2 michae2 commented Oct 31, 2024

In #93067 we added support for automatic collection of partial stats. This commit enables automatic collection of partial stats by default.

Epic: None

Release note (sql change): This commit turns on cluster setting sql.stats.automatic_partial_collection.enabled by default, which enables automatic collection of partial table stats. Partial table stats (i.e. those created with CREATE STATISTICS ... USING EXTREMES) scan the lower and upper ends of indexes to collect statistics outside the range covered by the previous full statistics collection.

In cockroachdb#93067 we added support for automatic collection of partial
stats. This commit enables automatic collection of partial stats by
default.

Release note (sql change): This commit turns on cluster setting
`sql.stats.automatic_partial_collection.enabled` by default, which
enables automatic collection of partial table stats. Partial table stats
(i.e. those created with `CREATE STATISTICS ... USING EXTREMES`) scan the
lower and upper ends of indexes to collect statistics outside the range
covered by the previous full statistics collection.
@michae2 michae2 requested review from a team, mgartner and rytaft October 31, 2024 18:02
@blathers-crl
Copy link
Copy Markdown

blathers-crl bot commented Oct 31, 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 3 of 3 files at r1, all commit messages.
Reviewable status: :shipit: complete! 1 of 0 LGTMs obtained (waiting on @mgartner)

@michae2
Copy link
Copy Markdown
Collaborator Author

michae2 commented Oct 31, 2024

bors r=rytaft

@craig
Copy link
Copy Markdown
Contributor

craig bot commented Oct 31, 2024

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.

3 participants