Github user omalley commented on a diff in the pull request: https://github.com/apache/orc/pull/299#discussion_r208392855 --- Diff: java/core/src/java/org/apache/orc/impl/ColumnStatisticsImpl.java --- @@ -543,35 +551,73 @@ public void reset() { super.reset(); minimum = null; maximum = null; + isLowerBoundSet = false; + isUpperBoundSet = false; sum = 0; } @Override public void updateString(Text value) { if (minimum == null) { - maximum = minimum = new Text(value); + if(value.getLength() > MAX_BYTES_RECORDED) { + minimum = truncateLowerBound(value); + maximum = truncateUpperBound(value); + isLowerBoundSet = true; + isUpperBoundSet = true; + } else { + maximum = minimum = new Text(value); + } } else if (minimum.compareTo(value) > 0) { - minimum = new Text(value); + if(value.getLength() > MAX_BYTES_RECORDED) { + minimum = truncateLowerBound(value); + isLowerBoundSet = true; + }else { + minimum = new Text(value); --- End diff -- This needs to set isLowerBoundSet = false.
---