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

2019-06-07 Thread GitBox
walterddr commented on issue #1011: [CALCITE-2721] Support parsing of DOT + MEMBER_FUNCTION URL: https://github.com/apache/calcite/pull/1011#issuecomment-499966495 @zhztheplayer I already have an experiment on CALCITE-2906 and seems like the concern is less obvious after I rebased with lat

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

2019-05-30 Thread GitBox
walterddr commented on issue #1011: [CALCITE-2721] Support parsing of DOT + MEMBER_FUNCTION URL: https://github.com/apache/calcite/pull/1011#issuecomment-497536745 @michaelmior thanks for the reminder. I just rebased and updated the change. I think the discussion has pretty much finalized.

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

2019-03-10 Thread GitBox
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 loo