wangxlong commented on a change in pull request #1795:
URL: https://github.com/apache/calcite/pull/1795#discussion_r493303937



##########
File path: core/src/main/java/org/apache/calcite/runtime/SqlFunctions.java
##########
@@ -1170,6 +1180,25 @@ private static ByteString binaryOperator(
     return new ByteString(result);
   }
 
+  /**
+   * Utility for bitwise function applied to byteString and long values.
+   *
+   * @param b0 The first byteString value operand of bitwise function.
+   * @param b1 The second long value operand of bitwise function.
+   * @param bitOp BitWise binary operator.
+   * @return ByteString after bitwise operation.
+   */
+  private static ByteString binaryOperator(
+      ByteString b0, long b1, BinaryOperator<Byte> bitOp) {
+    final byte[] bytes0 = b0.getBytes();
+
+    final byte[] result = new byte[bytes0.length];

Review comment:
       Yes, Updated.




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


Reply via email to