You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
keep the DataFusion code base clean and easy to contribute to and modify
Understand what features are covered and what are not
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
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:
Hi @alamb, I looked into this and it seems displayable.rs was already migrated with #8246, there is no such file anymore in the test folder. I think this issue can be closed as resolved.
Hi @alamb, I looked into this and it seems displayable.rs was already migrated with #8246, there is no such file anymore in the test folder. I think this issue can be closed as resolved.
Thank you @simicd -- I think you are right. Closing!
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/displayable.rs to
displayable.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: