Pouria Pirzadeh has uploaded a new change for review. https://asterix-gerrit.ics.uci.edu/329
Change subject: PLEASE EDIT to provide a meaningful commit message! ...................................................................... PLEASE EDIT to provide a meaningful commit message! The following commits from your working branch will be included: commit fe34e73c0f00c65300f7ffb2fd6186cabb1d33f9 Author: pouria.pirza...@gmail.com <pouria.pirza...@gmail.com> Date: Tue Jul 28 11:37:16 2015 -0700 NKC fix for int64 Change-Id: I60b3ec3f15921093b1fa11ac9fc157b7930f4e4d --- M hyracks/hyracks-dataflow-common/src/main/java/edu/uci/ics/hyracks/dataflow/common/data/normalizers/Integer64NormalizedKeyComputerFactory.java 1 file changed, 2 insertions(+), 2 deletions(-) git pull ssh://asterix-gerrit.ics.uci.edu:29418/hyracks refs/changes/29/329/1 diff --git a/hyracks/hyracks-dataflow-common/src/main/java/edu/uci/ics/hyracks/dataflow/common/data/normalizers/Integer64NormalizedKeyComputerFactory.java b/hyracks/hyracks-dataflow-common/src/main/java/edu/uci/ics/hyracks/dataflow/common/data/normalizers/Integer64NormalizedKeyComputerFactory.java index bcff79f..1608327 100644 --- a/hyracks/hyracks-dataflow-common/src/main/java/edu/uci/ics/hyracks/dataflow/common/data/normalizers/Integer64NormalizedKeyComputerFactory.java +++ b/hyracks/hyracks-dataflow-common/src/main/java/edu/uci/ics/hyracks/dataflow/common/data/normalizers/Integer64NormalizedKeyComputerFactory.java @@ -34,7 +34,7 @@ public int normalize(byte[] bytes, int start, int length) { long value = LongPointable.getLong(bytes, start); int highValue = (int) (value >> 32); - if (highValue > 0) { + if (value > Integer.MAX_VALUE) { /** * larger than Integer.MAX */ @@ -42,7 +42,7 @@ highNmk >>= 2; highNmk |= POSTIVE_LONG_MASK; return highNmk; - } else if (highValue == 0) { + } else if (value >=0 && value <= Integer.MAX_VALUE) { /** * smaller than Integer.MAX but >=0 */ -- To view, visit https://asterix-gerrit.ics.uci.edu/329 To unsubscribe, visit https://asterix-gerrit.ics.uci.edu/settings Gerrit-MessageType: newchange Gerrit-Change-Id: I60b3ec3f15921093b1fa11ac9fc157b7930f4e4d Gerrit-PatchSet: 1 Gerrit-Project: hyracks Gerrit-Branch: master Gerrit-Owner: Pouria Pirzadeh <pouria.pirza...@gmail.com>