Laszlo Bodor created HIVE-19131:

             Summary: DecimalColumnStatsMergerTest comparison review
                 Key: HIVE-19131
             Project: Hive
          Issue Type: Bug
            Reporter: Laszlo Bodor

DecimalColumnStatsMergerTest has a strange comparison logic, which needs to be 

Regarding low and high values, it uses compareTo with the same direction, which 
seems to be incorrect.


Decimal lowValue = aggregateData.getLowValue() != null && 
(aggregateData.getLowValue().compareTo(newData.getLowValue()) > 0) ? 
aggregateData .getLowValue() : newData.getLowValue(); 
aggregateData.setLowValue(lowValue); Decimal highValue = 
aggregateData.getHighValue() != null && 
(aggregateData.getHighValue().compareTo(newData.getHighValue()) > 0) ? 
aggregateData .getHighValue() : newData.getHighValue();


This message was sent by Atlassian JIRA

Reply via email to