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
