[multistage] Handle Integer.MIN_VALUE in hashCode based FieldSelectionKeySelector

ankitsultana · ankitsultana · commit e5857556a259 · 2023-04-12T13:51:11.000-07:00
diff --git a/pinot-query-planner/src/main/java/org/apache/pinot/query/planner/partitioning/FieldSelectionKeySelector.java b/pinot-query-planner/src/main/java/org/apache/pinot/query/planner/partitioning/FieldSelectionKeySelector.java
@@ -89,7 +89,7 @@ public int computeHash(Object[] input) {
     }
 
     // return a positive number because this is used directly to modulo-index
-    return Math.abs(hashCode);
+    return hashCode & Integer.MAX_VALUE;
   }
 
   @Override

Original file line number	Diff line number	Diff line change
`@@ -89,7 +89,7 @@ public int computeHash(Object[] input) {`
`89`	`89`	`}`
`90`	`90`
`91`	`91`	`// return a positive number because this is used directly to modulo-index`
`92`		`- return Math.abs(hashCode);`
	`92`	`+ return hashCode & Integer.MAX_VALUE;`
`93`	`93`	`}`
`94`	`94`
`95`	`95`	`@Override`