You are viewing a plain text version of this content. The canonical link for it is here.
Posted to issues@calcite.apache.org by "Stamatis Zampetakis (Jira)" <ji...@apache.org> on 2020/11/20 09:34:00 UTC

[jira] [Commented] (CALCITE-4410) Expose the type of internal sql node in Planner

    [ https://issues.apache.org/jira/browse/CALCITE-4410?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17236023#comment-17236023 ] 

Stamatis Zampetakis commented on CALCITE-4410:
----------------------------------------------

Are you proposing to add a new method in the {{Planner}} interface? 

The main goal of the {{Planner}} interface is to be simple and cover the most common needs of the clients. From my perspective also the existing {{Planner#validateAndGetType}} does not fit very well in this interface so I am not sure if it was a good idea to add in the first place. Worth noting that in our current codebase the method is not called by anyone.

Can you share a bit more about your use-case to see what you would like to achieve?

> Expose the type of internal sql node in Planner
> -----------------------------------------------
>
>                 Key: CALCITE-4410
>                 URL: https://issues.apache.org/jira/browse/CALCITE-4410
>             Project: Calcite
>          Issue Type: Improvement
>          Components: core
>    Affects Versions: 1.26.0
>            Reporter: Will Yu
>            Assignee: Will Yu
>            Priority: Minor
>
> Currently in Planner interface, there is only possible to fetch the *RelDataType* of the whole SqlNode through *validateAndGetType*, but really hard to fetch the type of one component in the SqlNode, e.g. one of the statement in the select list.
> Currently, we worked it around by duplicate some code in Calcite, which is not a good practice. It would be helpful to directly expose the *SqlValidator::getValidatedNodeType* through Planner.



--
This message was sent by Atlassian Jira
(v8.3.4#803005)