Skip to content

Improve formatting of unnest in plan printer#11630

Merged
sopel39 merged 1 commit intotrinodb:masterfrom
raunaqmorarka:unnest-print
Mar 23, 2022
Merged

Improve formatting of unnest in plan printer#11630
sopel39 merged 1 commit intotrinodb:masterfrom
raunaqmorarka:unnest-print

Conversation

@raunaqmorarka
Copy link
Copy Markdown
Member

Description

When no symbols are replicated from unnest, the printed plan
currently shows
"Unnest[replicate=, unnest=names:array(varchar(25))]"
This change improves it to
"Unnest[unnest=names:array(varchar(25))]"

Is this change a fix, improvement, new feature, refactoring, or other?

improvement

Is this a change to the core query engine, a connector, client library, or the SPI interfaces? (be specific)

query planner

How would you describe this change to a non-technical end user or system administrator?

Improve formatting of unnest in plan printer

Related issues, pull requests, and links

Documentation

(x) No documentation is needed.
( ) Sufficient documentation is included in this PR.
( ) Documentation PR is available with #prnumber.
( ) Documentation issue #issuenumber is filed, and can be handled later.

Release notes

(x) No release notes entries required.
( ) Release notes entries required with the following suggested text:

When no symbols are replicated from unnest, the printed plan
currently shows
"Unnest[replicate=, unnest=names:array(varchar(25))]"
This change improves it to
"Unnest[unnest=names:array(varchar(25))]"
@cla-bot cla-bot bot added the cla-signed label Mar 23, 2022
@raunaqmorarka raunaqmorarka requested a review from sopel39 March 23, 2022 09:37
@sopel39 sopel39 merged commit 7db9ad0 into trinodb:master Mar 23, 2022
@github-actions github-actions bot added this to the 375 milestone Mar 23, 2022
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