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 20:16:40 UTC

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

LakshSingla edited a comment 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`. From the Calcite's docs. 
   ```
   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. (Can be altered to whatever makes more 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