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.

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

release-2.1: sql: remove EXPLAIN EXECUTE #30725

Merged
merged 1 commit into from
Sep 27, 2018

Conversation

RaduBerinde
Copy link
Member

Backport 1/1 commits from #30700.

/cc @cockroachdb/release


Removing the syntax and the code for EXPLAIN EXECUTE. Currently this
does not work property with the optimizer and shows the heuristic
plan. See #30409 for more context.

Closes #30409.

Release note (sql change): EXECUTE is no longer an explainable
statement (i.e. EXPLAIN EXECUTE is not valid anymore). As an
alternative, it is possible to PREPARE ... AS EXPLAIN ... and then
execute this prepared statement to see the plan for a prepared query.

Removing the syntax and the code for `EXPLAIN EXECUTE`. Currently this
does not work property with the optimizer and shows the heuristic
plan. See cockroachdb#30409 for more context.

Closes cockroachdb#30409.

Release note (sql change): `EXECUTE` is no longer an explainable
statement (i.e. `EXPLAIN EXECUTE` is not valid anymore). As an
alternative, it is possible to `PREPARE ... AS EXPLAIN ...` and then
execute this prepared statement to see the plan for a prepared query.
@RaduBerinde RaduBerinde requested review from knz and a team September 27, 2018 16:33
@RaduBerinde RaduBerinde requested a review from a team as a code owner September 27, 2018 16:33
@RaduBerinde RaduBerinde requested review from a team September 27, 2018 16:33
@cockroach-teamcity
Copy link
Member

This change is Reviewable

@RaduBerinde RaduBerinde merged commit c947912 into cockroachdb:release-2.1 Sep 27, 2018
@RaduBerinde RaduBerinde deleted the backport2.1-30700 branch September 28, 2018 13:49
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.

None yet

3 participants