[GitHub] [calcite] beyond1920 commented on a diff in pull request #2606: [CALCITE-4865] Extends Table function to support Polymorphic Table Function (PTF)

[GitBox <gi...@apache.org>]

beyond1920 commented on code in PR #2606:
URL: https://github.com/apache/calcite/pull/2606#discussion_r906838046


##########
core/src/main/java/org/apache/calcite/sql/SqlTableFunction.java:
##########
@@ -30,4 +32,15 @@ public interface SqlTableFunction {
    * @return strategy to infer the row type of a call to this function
    */
   SqlReturnTypeInference getRowTypeInference();
+
+  /**
+   * Returns the table parameter characteristics for <code>ordinal</code>th argument to this
+   * table function.

Review Comment:
   Good catch.
   It would return null if the ordinal Nth argument is not table parameters or the original  is < 0 or >= the number of parameters.
   I've updated the comments of the function.



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