[GitHub] drill pull request: Map variance() and stddev() to var_samp() and ...

2016-03-21 Thread minji-kim
Github user minji-kim commented on a diff in the pull request: https://github.com/apache/drill/pull/437#discussion_r56927626 --- Diff: exec/java-exec/src/test/java/org/apache/drill/TestAggregationQueries.java --- @@ -0,0 +1,53 @@ +/** + * Licensed to the Apache Software Fou

[GitHub] drill pull request: Map variance() and stddev() to var_samp() and ...

2016-03-21 Thread jacques-n
Github user jacques-n commented on a diff in the pull request: https://github.com/apache/drill/pull/437#discussion_r56920816 --- Diff: exec/java-exec/src/test/java/org/apache/drill/TestAggregationQueries.java --- @@ -0,0 +1,53 @@ +/** + * Licensed to the Apache Software Fou

[GitHub] drill pull request: Map variance() and stddev() to var_samp() and ...

2016-03-21 Thread minji-kim
GitHub user minji-kim opened a pull request: https://github.com/apache/drill/pull/437 Map variance() and stddev() to var_samp() and stddev_samp() respectiv… …ely, and allow for DrillReduceAggregateRule to apply to them Right now, variance() and stddev() is not mapped to