sql: disallow creating partial stats by default #98500
Conversation
|
This change is |
michae2
left a comment
michae2
left a comment
There was a problem hiding this comment.
(Looks like some logic tests need to be rewritten.)
Reviewed 5 of 5 files at r1, all commit messages.
Reviewable status:complete! 1 of 0 LGTMs obtained (waiting on @mgartner)
pkg/sql/logictest/testdata/logic_test/distsql_stats line 2008 at r1 (raw file):
statement ok SET enable_create_stats_using_extremes = on
I think this will also need to be enabled in pkg/sql/opt/exec/execbuilder/testdata/partial_stats.
This commit adds the `enable_create_stats_using_extremes` session setting which allows users to run `CREATE STATISTICS .. USING EXTREMES` when enabled. It is disabled by default. This commit also presents a "feature not supported" error when a user tries to create partial statistics with a `WHERE` clause. This feature has not yet been implemented. Epic: CRDB-19449 Release note: None
mgartner
force-pushed
the
95233-partial-stats-settings
branch
from
80383de to
240ad78
Compare
mgartner
left a comment
mgartner
left a comment
There was a problem hiding this comment.
Reviewable status:
pkg/sql/logictest/testdata/logic_test/distsql_stats line 2008 at r1 (raw file):
Previously, michae2 (Michael Erickson) wrote…
I think this will also need to be enabled in
pkg/sql/opt/exec/execbuilder/testdata/partial_stats.
It's actually not needed because we inject partial stats there instead of running CREATE STATISTICS .. USING EXTREMES.
|
TFTRs! bors r+ |
|
Build failed (retrying...): |
|
Build succeeded: |
3 participants
This commit adds the
enable_create_stats_using_extremessessionsetting which allows users to run
CREATE STATISTICS .. USING EXTREMESwhen enabled. It is disabled by default.
This commit also presents a "feature not supported" error when a user
tries to create partial statistics with a
WHEREclause. This featurehas not yet been implemented.
Epic: CRDB-19449
Fixes #95233
Release note: None