-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/16829/
-----------------------------------------------------------

(Updated Jan. 16, 2014, 9:48 p.m.)


Review request for hive.


Changes
-------

* Rewrote if condition for better readability.
* Took care of red marks.
* Using HiveDecimal to take care of non-integral argument in sum.
* Added test for non-integral argument.


Bugs: HIVE-6192
    https://issues.apache.org/jira/browse/HIVE-6192


Repository: hive-git


Description
-------

optimize sum(1) query so that it could be answered from metadata using stats


Diffs (updated)
-----

  ql/src/java/org/apache/hadoop/hive/ql/optimizer/StatsOptimizer.java 75390e7 
  ql/src/test/queries/clientpositive/metadata_only_queries.q 7cbd148 
  ql/src/test/results/clientpositive/metadata_only_queries.q.out b6d149a 

Diff: https://reviews.apache.org/r/16829/diff/


Testing
-------

Added test in metadata_only_queries.q which contains other similar tests.


Thanks,

Ashutosh Chauhan

Reply via email to