You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@druid.apache.org by GitBox <gi...@apache.org> on 2021/11/11 14:02:45 UTC

[GitHub] [druid] LakshSingla commented on pull request #11908: Improve the output of SQL explain message

LakshSingla commented on pull request #11908:
URL: https://github.com/apache/druid/pull/11908#issuecomment-966326496


   We can support both the formats simultaneously in Calcite's syntax using `SqlExplainLevel` and `SqlExplain.Depth`. I have added. 
   ```
   explain:
         EXPLAIN PLAN
         [ WITH TYPE | WITH IMPLEMENTATION | WITHOUT IMPLEMENTATION ]
         [ EXCLUDING ATTRIBUTES | INCLUDING [ ALL ] ATTRIBUTES ]
         [ AS JSON | AS XML | AS DOT ]
         FOR { query | insert | update | merge | delete }
   ```
   So potentially,
   `EXPLAIN PLAN FOR {query}` can give older output and say `EXPLAIN PLAN EXCLUDING ATTRIBUTES {query}` can give the newer one. (Whichever makes mores sense semantically)


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: commits-unsubscribe@druid.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org



---------------------------------------------------------------------
To unsubscribe, e-mail: commits-unsubscribe@druid.apache.org
For additional commands, e-mail: commits-help@druid.apache.org