This is an automated email from the ASF dual-hosted git repository.
xudong963 pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/arrow-datafusion.git
The following commit(s) were added to refs/heads/master by this push:
new 5d69fa33b MINOR: Add note about sqllogictest to contributor guide
(#4469)
5d69fa33b is described below
commit 5d69fa33b84e9d7b54477ec0f464cbac69825157
Author: Andrew Lamb <[email protected]>
AuthorDate: Fri Dec 2 10:01:41 2022 -0500
MINOR: Add note about sqllogictest to contributor guide (#4469)
* docs: Add note about sqllogictest to contributor guide
* Update docs/source/contributor-guide/index.md
---
docs/source/contributor-guide/index.md | 7 +++++++
1 file changed, 7 insertions(+)
diff --git a/docs/source/contributor-guide/index.md
b/docs/source/contributor-guide/index.md
index 77ef01bc0..38fcd70a9 100644
--- a/docs/source/contributor-guide/index.md
+++ b/docs/source/contributor-guide/index.md
@@ -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
