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

sql: decrease vectorize_row_count_threshold to 0 #9553

Closed
jseldess opened this issue Jan 29, 2021 · 0 comments · Fixed by #10240
Closed

sql: decrease vectorize_row_count_threshold to 0 #9553

jseldess opened this issue Jan 29, 2021 · 0 comments · Fixed by #10240
Assignees
@ianjevans
Labels
C-product-change P-1 High priority; must be done this release
Milestone
21.1

Comments

@jseldess
Copy link
Contributor

PR: cockroachdb/cockroach#55713

From release notes:

The default value for vectorize_row_count_threshold setting has been decreased from 1000 to 0, meaning that from now on CockroachDB will always use the vectorized engine for all supported queries regardless of the row estimate (unless vectorize=off is set). [#55713][#55713] {% comment %}doc{% endcomment %}
@jseldess jseldess added this to the 21.1 milestone Jan 29, 2021
@jseldess jseldess added the A-sql label Jan 29, 2021
@ericharmeling ericharmeling added the P-1 High priority; must be done this release label Jan 29, 2021
@ianjevans ianjevans assigned ianjevans and unassigned ericharmeling Feb 4, 2021
ianjevans pushed a commit that referenced this issue Apr 6, 2021
Fixes #9553.
305cbcc
@ianjevans ianjevans mentioned this issue Apr 6, 2021
Merged
ianjevans pushed a commit that referenced this issue Apr 12, 2021
Update EXPLAIN and EXPLAIN ANALYZE topics and output for 21.1 (#1…
95cad3c 
…0240)

* New EXPLAIN ANALYZE PLAN statement.

* Pass to update all EXPLAIN output.

* Fixes #10127 estimated row count in all operators of EXPLAIN.
Fixes #10020, clarify full vs local distributed execution plans.

* Updates to EXPLAIN ANALYZE.

* Adding new demo workload include.

* Adding new statistics, and renaming KV tuple -> KV row.

* Fixes #9582.

* Fixes #9154.

* Fixes #9109.

* Backport fix for #10020.

* Removing insecure performance tuning tutorial.

* Addressed Radu's review comments.

* Fixes #9553.

* Refactoring to use demo cluster with locality flags.

* Incorporated Andy's review comments.

* Incorporated Eric's feedback.

* Generated new `EXPLAIN ANALYZE` diagram.

* Fixes #10262
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@ianjevans ianjevans

Labels
C-product-change P-1 High priority; must be done this release
Projects
None yet
Milestone
21.1
Development

Successfully merging a pull request may close this issue.

Update EXPLAIN and EXPLAIN ANALYZE topics and output for 21.1 cockroachdb/docs
3 participants
@ianjevans @jseldess @ericharmeling