DonnyZone commented on issue #1593: [CALCITE-3513] Unify TableFunction 
implementor's NullPolicy and its behavior
URL: https://github.com/apache/calcite/pull/1593#issuecomment-558466787
 
 
   `ScalarFunctionImpl` determines NullPolicy according to annotations, while 
`TableFunctionImpl` sets `NullPolicy=ANY` by default for all cases.
   However, according to the javadoc in `TableFunction`, `null` arguments are 
allowed.
   ```
     /**
      * Returns the row type of the table yielded by this function when
      * applied to given arguments. Only literal arguments are passed,
      * non-literal are replaced with default values (null, 0, false, etc).
      *
      * @param arguments arguments of a function call (only literal arguments
      *                  are passed, nulls for non-literal ones)
      * @return element type of the table (e.g. {@code Object[].class})
      */
     Type getElementType(List<Object> arguments);
   ```

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

Reply via email to