Skip to content

Commit

Permalink
MINOR: Add note about sqllogictest to contributor guide (#4469)
Browse files Browse the repository at this point in the history
* docs: Add note about sqllogictest to contributor guide

* Update docs/source/contributor-guide/index.md
  • Loading branch information
alamb authored Dec 2, 2022
1 parent b229e0f commit 5d69fa3
Showing 1 changed file with 7 additions and 0 deletions.
7 changes: 7 additions & 0 deletions docs/source/contributor-guide/index.md
Original file line number Diff line number Diff line change
Expand Up @@ -134,6 +134,13 @@ cargo test -p datafusion --tests sql_integration

One very important test is the [sql_integration](../../../datafusion/core/tests/sql_integration.rs) test which validates DataFusion's ability to run a large assortment of SQL queries against an assortment of data setups.

### sqllogictests Tests

The [sqllogictests](../../../datafusion/core/tests/sqllogictests) also validate DataFusion SQL against an assortment of data setups.

Data Driven tests have many benefits including being easier to write and maintain. We are in the process of [migrating sql_integration tests](https://github.com/apache/arrow-datafusion/issues/4460) and encourage
you to add new tests using sqllogictests if possible.

### SQL / Postgres Integration Tests

The [integration-tests](../../../integration-tests) directory contains a harness that runs certain queries against both postgres and datafusion and compares results
Expand Down

0 comments on commit 5d69fa3

Please sign in to comment.