----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/32901/ -----------------------------------------------------------
(Updated April 7, 2015, 9:48 p.m.) Review request for hive, Ashutosh Chauhan and Prasanth_J. Changes ------- Created new DateColumnStatsData in hive_metastore.thrift, which will be used for the date stats. Also updated describe formatter/UpdateStatsTask to handle Date column stats. I've removed the generated Thrift code from diff to make this more readable. Bugs: HIVE-10226 https://issues.apache.org/jira/browse/HIVE-10226 Repository: hive-git Description ------- Re-use the long stats for Date column stats, using the days since epoch value as the long value. Diffs (updated) ----- metastore/if/hive_metastore.thrift 57bce0c metastore/src/java/org/apache/hadoop/hive/metastore/StatObjectConverter.java 475883b metastore/src/model/org/apache/hadoop/hive/metastore/model/MPartitionColumnStatistics.java 1666dc3 metastore/src/model/org/apache/hadoop/hive/metastore/model/MTableColumnStatistics.java bce9f0f ql/src/java/org/apache/hadoop/hive/ql/exec/ColumnStatsTask.java 0c46b00 ql/src/java/org/apache/hadoop/hive/ql/exec/ColumnStatsUpdateTask.java b85282c ql/src/java/org/apache/hadoop/hive/ql/metadata/formatting/MetaDataFormatUtils.java 1662696 ql/src/java/org/apache/hadoop/hive/ql/udf/generic/GenericUDAFComputeStats.java 363039b ql/src/test/queries/clientpositive/compute_stats_date.q PRE-CREATION ql/src/test/results/clientpositive/compute_stats_date.q.out PRE-CREATION Diff: https://reviews.apache.org/r/32901/diff/ Testing ------- Thanks, Jason Dere