Home | About | Sematext search-lucene.com search-hadoop.com
 Search Hadoop and all its subprojects:

Switch to Threaded View
Drill >> mail # dev >> Review Request: Add RunningAggregate Reference Implementation


Copy link to this message
-
Re: Review Request: Add RunningAggregate Reference Implementation

-----------------------------------------------------------
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