----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/10209/#review19584 -----------------------------------------------------------
Did you make any tests for this to validate that it functions as expected? sandbox/prototype/exec/ref/src/main/java/org/apache/drill/exec/ref/eval/EvaluatorTypes.java <https://reviews.apache.org/r/10209/#comment40442> I had made a coment here before that it might be worthwhile to extract out eval() and clear() rather than have a separate runningEval(). Did you see that comment and disagree or never see that comment? sandbox/prototype/exec/ref/src/main/java/org/apache/drill/exec/ref/rops/RunningAggregateROP.java <https://reviews.apache.org/r/10209/#comment40441> Small nit... Boundary is spelled with an 'a'. There are a bunch of variables in here that have the misspelling. - Jacques Nadeau On April 2, 2013, 8:26 p.m., Timothy Chen wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/10209/ > ----------------------------------------------------------- > > (Updated April 2, 2013, 8:26 p.m.) > > > Review request for drill. > > > Description > ------- > > Added RunningAggregate Ref Impl > > > Diffs > ----- > > > 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 > > Diff: https://reviews.apache.org/r/10209/diff/ > > > Testing > ------- > > > Thanks, > > Timothy Chen > >
