robinyqiu commented on a change in pull request #13364:
URL: https://github.com/apache/beam/pull/13364#discussion_r546869263
##########
File path:
sdks/java/extensions/sql/src/main/java/org/apache/beam/sdk/extensions/sql/impl/transform/BeamBuiltinAggregations.java
##########
@@ -397,6 +399,93 @@ public Long extractOutput(Long accum) {
* <p>Note: null values are ignored when mixed with non-null values.
* (https://issues.apache.org/jira/browse/BEAM-10379)
*/
+ // static class BitAnd<T extends Number> extends CombineFn<T, Long, Long> {
Review comment:
This PR and https://github.com/apache/beam/pull/13391 both have change
to add back the already deleted lines. I think this is unintended? Could you
please remove these from these 2 PRs?
----------------------------------------------------------------
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]
