Skip to content

implement kurtosis udaf - #12613

Closed
jatin510 wants to merge 2 commits into
apache:mainfrom
jatin510:feature/12250-kurtosis
Closed

implement kurtosis udaf#12613
jatin510 wants to merge 2 commits into
apache:mainfrom
jatin510:feature/12250-kurtosis

Conversation

@jatin510

Copy link
Copy Markdown
Contributor

Which issue does this PR close?

Closes#12250

Rationale for this change

I followed the changes in PR: #12273

What changes are included in this PR?

Are these changes tested?

yes

Are there any user-facing changes?

@github-actionsgithub-actionsBot added documentation Improvements or additions to documentation sqllogictest SQL Logic Tests (.slt) proto Related to proto crate functions Changes to functions implementation labels Sep 25, 2024
@Weijun-H

Copy link
Copy Markdown
Member

Given #12357 we may want to move this function to some other repo rather than the core

FYI #12357

@alamb

Copy link
Copy Markdown
Contributor

Thank you @jatin510 .

As @Weijun-H says, is there any chance you can retarget this PR to https://github.com/datafusion-contrib/datafusion-functions-extra instead?

@alamb
alamb marked this pull request as draft September 25, 2024 15:42
@jatin510

Copy link
Copy Markdown
ContributorAuthor

sure, I will make the changes.

@alamb

Copy link
Copy Markdown
Contributor

Closing as this PR is going to be retargeted

@alambalamb closed this Sep 26, 2024
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Labels

documentationImprovements or additions to documentationfunctionsChanges to functions implementationprotoRelated to proto cratesqllogictestSQL Logic Tests (.slt)

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Support kurtosis(x) in Aggregation function

3 participants

@jatin510@Weijun-H@alamb