zhztheplayer commented on a change in pull request #1011: [CALCITE-2721]
Support parsing of DOT + MEMBER_FUNCTION
URL: https://github.com/apache/calcite/pull/1011#discussion_r264614447
##########
File path: core/src/test/java/org/apache/calcite/sql/parser/SqlParserTest.java
##########
@@ -7385,6 +7387,24 @@ public void subTestIntervalSecondFailsValidation() {
"(?s).*Encountered \"from\" at .*");
}
+ /**
+ * Tests that applying member function of a specific type as a suffix
function
+ */
+ @Test public void testMemberFunction() {
+ check("SELECT myColumn.func(a, b) FROM tbl",
Review comment:
Thank you for your explanation, I too think it's hard to distinguish them
now.
As for the array element part, do you already have a plan? I guess it maybe
better to somehow move `<DOT> RowExpressionExtension() ...` syntax code to
`Expression2()`, for keeping `Expression2b()` clean.
----------------------------------------------------------------
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:
[email protected]
With regards,
Apache Git Services
