ibzib commented on a change in pull request #12169: URL: https://github.com/apache/beam/pull/12169#discussion_r450527335
########## File path: sdks/java/extensions/sql/zetasql/src/main/java/org/apache/beam/sdk/extensions/sql/zetasql/translation/ConversionContext.java ########## @@ -33,7 +33,16 @@ private final ExpressionConverter expressionConverter; private final RelOptCluster cluster; private final QueryTrait trait; - private final Map<List<String>, ResolvedNode> sqlUDTVF; + // Pure-sql user-defined table-valued function can be resolved by Analyzer. Keeping the Review comment: I'm guessing the meaning of `Pure-sql` might not be clear to those who are unfamiliar with our plans to add cross-language UDF. ---------------------------------------------------------------- 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: us...@infra.apache.org