-
Notifications
You must be signed in to change notification settings - Fork 2k
Closed
Labels
good first issueGood for newcomersGood for newcomershelp wantedExtra attention is neededExtra attention is neededsqllogictestSQL Logic Tests (.slt)SQL Logic Tests (.slt)
Description
#4395 moved several initial tests from sql_integration into sqllogictests
Specifically, tests that were moved include:
- https://github.com/apache/arrow-datafusion/blob/master/datafusion/core/tests/sql/aggregates.rs --> https://github.com/apache/arrow-datafusion/blob/master/datafusion/core/tests/sqllogictests/test_files/aggregate.slt
- https://github.com/apache/arrow-datafusion/blob/master/datafusion/core/tests/sql/arrow_typeof.rs --> https://github.com/apache/arrow-datafusion/blob/master/datafusion/core/tests/sqllogictests/test_files/arrow_typeof.slt
The goal of this ticket is to:
- Review that all tests were moved (and none were added subsequently, for example)
- Port any remaining gaps
- Remove the
.rsfiles
Reactions are currently unavailable
Metadata
Metadata
Assignees
Labels
good first issueGood for newcomersGood for newcomershelp wantedExtra attention is neededExtra attention is neededsqllogictestSQL Logic Tests (.slt)SQL Logic Tests (.slt)