Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Define APPROX_COUNT_DISTINCT in yaml file for approximate aggregate functions #204

Merged
merged 1 commit into from
May 20, 2022

Conversation

JamesRTaylor
Copy link
Contributor

@JamesRTaylor JamesRTaylor commented May 19, 2022

Adds a new yaml file for approximate function implementations, starting with defining APPROX_COUNT_DISTINCT which can be found in many databases:
SQL Server/Azure
Oracle
Snowflake
Spark
BigQuery
Databricks
Dremio
VoltDB
AlibabaCloud
Druid
Vertica though they name it APPROXIMATE_COUNT_DISTINCT
PrestoDB/Trino though they name it APPROX_DISTINCT

@JamesRTaylor
Copy link
Contributor Author

Thanks for the review, @jacques-n. I've updated the PR.

Copy link
Contributor

@jacques-n jacques-n left a comment

Choose a reason for hiding this comment

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

LGTM, thanks!

@jacques-n jacques-n merged commit 8e206b9 into substrait-io:main May 20, 2022
rkondakov added a commit to rkondakov/substrait that referenced this pull request Nov 21, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants