[GitHub] [calcite] JiajunBernoulli commented on a diff in pull request #3270: [CALCITE-5779] Implicit column alias for single-column table function should work

["JiajunBernoulli (via GitHub)" <gi...@apache.org>]

JiajunBernoulli commented on code in PR #3270:
URL: https://github.com/apache/calcite/pull/3270#discussion_r1232923846


##########
core/src/test/java/org/apache/calcite/test/SqlValidatorTest.java:
##########
@@ -7598,6 +7598,18 @@ void testGroupExpressionEquivalenceParams() {
         .fails("Cannot call table function here: 'RAMP'");
   }
 
+  /** Test case for
+   * <a href="https://issues.apache.org/jira/browse/CALCITE-5779">[CALCITE-5779]
+   * Implicit column alias for single-column table function should work</a>. */
+  @Test void testTableFunctionSingleColumnAlias() {
+    final SqlValidatorFixture s = fixture()
+        .withOperatorTable(MockSqlOperatorTable.standard().extend());
+    s.withSql("select rmp from table(ramp(3)) as rmp").ok();

Review Comment:
   Should we need `SqlToRelConverterTest` or `QuidemTest` to confirm plan is right?



-- 
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.

To unsubscribe, e-mail: commits-unsubscribe@calcite.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org