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

(Updated June 14, 2014, 7:21 p.m.)


Review request for drill, Aditya Kishore, Mehant Baid, and Timothy Chen.


Changes
-------

Implemented review comments!


Repository: drill-git


Description
-------

Added implementation for Correlation coefficient calculation function.


Diffs (updated)
-----

  exec/java-exec/src/main/codegen/config.fmpp 5c0d03b 
  exec/java-exec/src/main/codegen/data/CorrelationTypes.tdd PRE-CREATION 
  exec/java-exec/src/main/codegen/data/CovarTypes.tdd PRE-CREATION 
  exec/java-exec/src/main/codegen/templates/CorrelationTypeFunctions.java 
PRE-CREATION 
  exec/java-exec/src/main/codegen/templates/CovarTypeFunctions.java 
PRE-CREATION 
  
exec/java-exec/src/test/java/org/apache/drill/exec/fn/impl/TestAggregateFunction.java
 ffb372d 
  exec/java-exec/src/test/resources/functions/test_covariance.json PRE-CREATION 

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


Testing
-------

Yes.
$mvn test -Dtest=TestAggregateFunction#testCovarianceCorrelation


Thanks,

Yash Sharma

Reply via email to