Skip to content
This repository was archived by the owner on May 9, 2024. It is now read-only.

Added QUANTILE agg function to Calcite #657

Merged
merged 1 commit into from
Sep 20, 2023
Merged

Added QUANTILE agg function to Calcite #657

merged 1 commit into from
Sep 20, 2023

Conversation

AndreyPavlenko
Copy link
Contributor

No description provided.

ienkovich
ienkovich reviewed Aug 31, 2023
View reviewed changes
Copy link
Contributor

@ienkovich ienkovich left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Is this declaration good enough to handle interpolation methods supported by QUANTILE? Also, QUANTILE supports datetime data types.

@AndreyPavlenko
Copy link
Contributor Author

Added the interpolation parameter, datetime data is also supported, the test passes - https://github.com/modin-project/modin/pull/6528/files#diff-924b045aa089c05b3a717bc9cea19dc8581e2b8351fcc3c0568a54a27ce8a146R1284

@ienkovich ienkovich merged commit 1a8c101 into intel:main Sep 20, 2023
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Reviewers

@ienkovich ienkovich ienkovich left review comments

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@AndreyPavlenko @ienkovich