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.1: sql: switch table and index in EXPLAIN (DISTSQL) diagrams #9154

Closed
lnhsingh opened this issue Dec 10, 2020 · 0 comments · Fixed by #10240
Closed

release-20.1: sql: switch table and index in EXPLAIN (DISTSQL) diagrams #9154

lnhsingh opened this issue Dec 10, 2020 · 0 comments · Fixed by #10240
Assignees
@ianjevans
Labels
C-product-change P-1 High priority; must be done this release
Milestone
20.1

Comments

@lnhsingh
Copy link
Contributor

PR: cockroachdb/cockroach#56396

From release notes:

Table names are now listed before index names in EXPLAIN (DISTSQL) diagram output. Previously, the diagrams used index@table, and now they use table@index. [#56396][#56396] {% comment %}doc{% endcomment %}
@lnhsingh lnhsingh added this to the 20.1.x milestone Dec 10, 2020
@ericharmeling ericharmeling added A-sql P-1 High priority; must be done this release labels Jan 7, 2021
@ianjevans ianjevans assigned ianjevans and unassigned ericharmeling Feb 4, 2021
ianjevans pushed a commit that referenced this issue Apr 5, 2021
Fixes #9154.
82388cb
This was referenced Apr 5, 2021
Closed
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
@jseldess jseldess modified the milestones: 20.1.x, 20.1 May 25, 2021
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
20.1
Development

Successfully merging a pull request may close this issue.

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