Skip to content

remove mentions of DruidQueryRel from docs#13033

Merged
vogievetsky merged 3 commits intoapache:masterfrom
vogievetsky:fix_explain_docs
Sep 6, 2022
Merged

remove mentions of DruidQueryRel from docs#13033
vogievetsky merged 3 commits intoapache:masterfrom
vogievetsky:fix_explain_docs

Conversation

@vogievetsky
Copy link
Contributor

After #12936 users will never see the DruidQueryRel based plans so the docs need to be updated to the new (and simpler) explain format

vogievetsky and others added 2 commits September 6, 2022 10:56
Co-authored-by: Victoria Lim <vtlim@users.noreply.github.com>
Co-authored-by: Victoria Lim <vtlim@users.noreply.github.com>
Copy link
Member

@vtlim vtlim left a comment

Choose a reason for hiding this comment

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

LGTM 🦖

@vogievetsky vogievetsky merged commit 897689c into apache:master Sep 6, 2022
@vogievetsky vogievetsky deleted the fix_explain_docs branch September 6, 2022 20:37
vogievetsky added a commit to vogievetsky/druid that referenced this pull request Sep 7, 2022
* remove mentions of DruidQueryRel

* Update docs/querying/sql-translation.md

Co-authored-by: Victoria Lim <vtlim@users.noreply.github.com>

* Update docs/querying/sql-translation.md

Co-authored-by: Victoria Lim <vtlim@users.noreply.github.com>

Co-authored-by: Victoria Lim <vtlim@users.noreply.github.com>
@abhishekagarwal87 abhishekagarwal87 added this to the 24.0.0 milestone Sep 7, 2022
abhishekagarwal87 pushed a commit that referenced this pull request Sep 7, 2022
* remove mentions of DruidQueryRel

* Update docs/querying/sql-translation.md

Co-authored-by: Victoria Lim <vtlim@users.noreply.github.com>

* Update docs/querying/sql-translation.md

Co-authored-by: Victoria Lim <vtlim@users.noreply.github.com>

Co-authored-by: Victoria Lim <vtlim@users.noreply.github.com>

Co-authored-by: Victoria Lim <vtlim@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants