Move Covariance (Population) covar_pop to be a User Defined Aggregate Function

Open yyy1000 opened this issue 1 year ago • 2 comments

Which issue does this PR close?

Closes #10389 .

Rationale for this change

What changes are included in this PR?

Are these changes tested?

Are there any user-facing changes?

May 07 '24 23:05 yyy1000

I agree once we port over the tests this PR should be good to go 🙏

Would you prefer moving test in a separate PR or in this PR? I'm good either way. :)

May 08 '24 17:05 yyy1000

I agree once we port over the tests this PR should be good to go 🙏

Would you prefer moving test in a separate PR or in this PR? I'm good either way. :)

If the test can be verified one by one less than 5min, than it is fine for me to see them in one PR.

May 09 '24 03:05 jayzhan211

Thanks @yyy1000 and @jayzhan211

May 10 '24 12:05 alamb