robinyqiu commented on a change in pull request #13364:
URL: https://github.com/apache/beam/pull/13364#discussion_r546872697
##########
File path:
sdks/java/extensions/sql/zetasql/src/main/java/org/apache/beam/sdk/extensions/sql/zetasql/translation/impl/BeamBuiltinMethods.java
##########
@@ -70,4 +70,7 @@
public static final Method DATE_METHOD =
Types.lookupMethod(DateFunctions.class, "date", Integer.class,
Integer.class, Integer.class);
+
+ public static final Method LOGICAL_AND =
Review comment:
I think this is called reflectively in `SqlOperators.java` by its name
as a string? So we may not be able to remove.
----------------------------------------------------------------
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]
