Github user paul-rogers commented on a diff in the pull request:
https://github.com/apache/drill/pull/1070#discussion_r158164933
--- Diff:
exec/java-exec/src/main/java/org/apache/drill/exec/expr/fn/impl/XXHash.java ---
@@ -166,9 +164,7 @@ public static long hash64(double val, long seed){
}
public static long hash64(long start, long end, DrillBuf buffer, long
seed){
- if (BoundsChecking.BOUNDS_CHECKING_ENABLED) {
- buffer.checkBytes((int)start, (int)end);
- }
+ rangeCheck(buffer, (int)start, (int)end);
--- End diff --
Why are the arguments to `hash64` longs? The hash may be 64 bits, but Drill
does not support vectors above 2 GB (31 bits) in length, so the `start` and
`end` need only be `int`.
---
