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

Julian Hyde commented on CALCITE-4580:
--------------------------------------

Hey, look: [public List<Pair<SqlIdentifier, SqlNode>> 
SqlCreateForeignSchema.options()|https://github.com/apache/calcite/blob/4bc916619fd286b2c0cc4d5c653c96a68801d74e/core/src/main/java/org/apache/calcite/sql/ddl/SqlCreateForeignSchema.java#L108].

> Change the visibility for SqlNode constructor
> ---------------------------------------------
>
>                 Key: CALCITE-4580
>                 URL: https://issues.apache.org/jira/browse/CALCITE-4580
>             Project: Calcite
>          Issue Type: Task
>            Reporter: Konstantin Orlov
>            Priority: Minor
>              Labels: pull-request-available
>          Time Spent: 10m
>  Remaining Estimate: 0h
>
> Currently the constructor is package private. That forces developers 
> extending the parser to put all custom nodes under {{org.apache.calcite.sql}} 
> package (or at least create one abstract node {{MySqlNode}} extending 
> {{SqlNode}} and exposing the constructor). It would nice to have the 
> constructor exposed by {{SqlNode}} itself.



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

Reply via email to