----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/10209/ -----------------------------------------------------------
(Updated May 1, 2013, 6:19 p.m.) Review request for drill. Changes ------- Added unit tests, also fixes typo for variable names Description ------- Added RunningAggregate Ref Impl Diffs (updated) ----- sandbox/prototype/common/src/main/java/org/apache/drill/common/logical/data/LogicalOperator.java e2bda45 sandbox/prototype/common/src/main/java/org/apache/drill/common/logical/data/RunningAggregate.java 01df9a0 sandbox/prototype/exec/ref/src/main/java/org/apache/drill/exec/ref/eval/EvaluatorTypes.java 2cb65ff sandbox/prototype/exec/ref/src/main/java/org/apache/drill/exec/ref/eval/fn/agg/AggregatingWrapperEvaluator.java ab7358f sandbox/prototype/exec/ref/src/main/java/org/apache/drill/exec/ref/eval/fn/agg/CountAggregator.java 099c001 sandbox/prototype/exec/ref/src/main/java/org/apache/drill/exec/ref/eval/fn/agg/SumAggregator.java 63fe017 sandbox/prototype/exec/ref/src/main/java/org/apache/drill/exec/ref/rops/RunningAggregateROP.java PRE-CREATION sandbox/prototype/exec/ref/src/test/java/org/apache/drill/exec/ref/rops/RunningAggregateROPTest.java PRE-CREATION Diff: https://reviews.apache.org/r/10209/diff/ Testing ------- Thanks, Timothy Chen
