Skip to content

Deprecate EXPLAIN (TYPE DISTRIBUTED)#27626

Closed
chenjian2664 wants to merge 1 commit into
trinodb:masterfrom
chenjian2664:jack/explain-query-refactor
Closed

Deprecate EXPLAIN (TYPE DISTRIBUTED)#27626
chenjian2664 wants to merge 1 commit into
trinodb:masterfrom
chenjian2664:jack/explain-query-refactor

Conversation

@chenjian2664
Copy link
Copy Markdown
Contributor

@chenjian2664 chenjian2664 commented Dec 12, 2025

Description

follow up of #27434
Going to deprecate the EXPLAIN (TYPE DISTRIBUTED)

#27153 (comment)

Additional context and related issues

Release notes

( ) This is not user-visible or is docs only, and no release notes are required.
( ) Release notes are required. Please propose a release note for me.
(x) Release notes are required, with the following suggested text:

## Section
* Deprecate EXPLAIN type LOGICAL and DISTRIBUTED. Use EXPLAIN without a type clause, instead

@chenjian2664
Copy link
Copy Markdown
Contributor Author

does this line needs to be updated

ExplainType.Type planType = DISTRIBUTED;
?

@chenjian2664 chenjian2664 force-pushed the jack/explain-query-refactor branch from 5e94ddb to 5a8a2dd Compare December 12, 2025 02:03
Copy link
Copy Markdown
Member

@findepi findepi left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I don't think it's a good idea.

It's clear EXPLAIN has a type attribute

EXPLAIN (TYPE IO) ...
EXPLAIN (TYPE VALIDATE) ...

it's not user friendly when there is no TYPE option that corresponds to the default behavior

@chenjian2664
Copy link
Copy Markdown
Contributor Author

@findepi Thanks

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Development

Successfully merging this pull request may close these issues.

2 participants