Jim Apple has posted comments on this change.

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


Patch Set 2:

> We "were" able to expose them as built-in functions and i tested
 > them on a small sample data (TPCH) at that time. The problem we had
 > was that we were unable to generate tests at that time.

I see. Can you please upload the code that exposed them as built-in functions 
to this patch? All of the code should be reviewed in one patch.

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