You are viewing a plain text version of this content. The canonical link for it is here.

Posted to commits@calcite.apache.org by GitBox <gi...@apache.org> on 2019/03/10 23:42:45 UTC

[GitHub] [calcite] walterddr commented on issue #1011: [CALCITE-2721] Support parsing of DOT + MEMBER_FUNCTION

walterddr commented on issue #1011: [CALCITE-2721] Support parsing of DOT + MEMBER_FUNCTION
URL: https://github.com/apache/calcite/pull/1011#issuecomment-471366200
 
 
   #958 has been merged. thus I updated the `RowExpressionExtension` based on SimpleIdentifier approach. Please kindly take a look @zhztheplayer @julianhyde . 
   
   One problem remains: 
   * `tbl.col.bar` is parsed as: ``` `TBL`.`COL`.`BAR` ```
   * `tbl.foo(0).col.bar` is parsed as: ``` ((`TBL`.`FOO`(0).`COL`).`BAR`) ```
   
   I am planning to fix this on [CALCITE-2906](https://issues.apache.org/jira/browse/CALCITE-2906). 
   If we think we should only support compound identifier with member function I can also alter this PR to do so.

----------------------------------------------------------------
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.
 
For queries about this service, please contact Infrastructure at:
users@infra.apache.org


With regards,
Apache Git Services