-
Notifications
You must be signed in to change notification settings - Fork 1.2k
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
Port tests in aggregates.rs
to sqllogictest
#8194
Comments
@alamb I was starting to look into this, and I see an https://github.com/apache/arrow-datafusion/blob/main/datafusion/sqllogictest/test_files/aggregate.slt already existing. Can I assume that all tests that are still in https://github.com/apache/arrow-datafusion/issues/8194#:~:text=main/datafusion/core/tests/sql/aggregates.rs have not been migrated to SQL Tests? |
I think you'll need to check the tests individually to see if they have already been ported |
Leaving here as a reference the current Rust test lists:
|
I think we have ported all the tests that make sense here so claiming victory |
Is your feature request related to a problem or challenge?
Part of #6195. We are trying to
Part of doing so is having a single location for most test coverage so new tests can be added easily and people can follow the existing patterns easily
Describe the solution you'd like
Port the tests in https://github.com/apache/arrow-datafusion/blob/main/datafusion/core/tests/sql/aggregates.rs to
aggregates.slt
in https://github.com/apache/arrow-datafusion/tree/main/datafusion/sqllogictest/test_filesNote you can create catalogs and schema via SQL now
Catalogs: https://arrow.apache.org/datafusion/user-guide/sql/ddl.html#create-database
Schema: https://arrow.apache.org/datafusion/user-guide/sql/ddl.html#create-schema
Notes:
Describe alternatives you've considered
No response
Additional context
I think these are good first issues as they teach the contributor about the DataFusion codebase and tests, as well as see the end user apis in practice.
The text was updated successfully, but these errors were encountered: