Jim Apple has posted comments on this change.

Change subject: IMPALA-1766: Misc. statistical functions. Implemented aggregate 
corr().
......................................................................


Patch Set 2:

> This is the same code I tested. It was a part of my semester
 > project and I gave a demo showing how all these functions appear as
 > a default built-in aggregate functions in Impala, and also compared
 > the results for Median that I implemented with the approximate one
 > Impala already have.

Are you suggesting that the code, as written in ther version of the patch you 
have already uploaded, already exposes these functions as built-in aggregate 
functions usable in the impala-shell?

-- 
To view, visit http://gerrit.cloudera.org:8080/2534
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings

Gerrit-MessageType: comment
Gerrit-Change-Id: I7347c173ec4f80037d45dd463c17eb81ceef14a1
Gerrit-PatchSet: 2
Gerrit-Project: Impala
Gerrit-Branch: cdh5-trunk
Gerrit-Owner: Abdur Rafay <[email protected]>
Gerrit-Reviewer: Abdur Rafay
Gerrit-Reviewer: Jim Apple <[email protected]>
Gerrit-Reviewer: Skye Wanderman-Milne <[email protected]>
Gerrit-HasComments: No

Reply via email to