MINOR: Add note about sqllogictest to contributor guide#4469
MINOR: Add note about sqllogictest to contributor guide#4469xudong963 merged 2 commits intoapache:masterfrom
Conversation
|
cc @xudong963 |
|
|
||
| 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 |
There was a problem hiding this comment.
How about adding some comparison with Rust Integration Tests and encouraging adding logictestes as far as possible?
|
Benchmark runs are scheduled for baseline = b229e0f and contender = 5d69fa3. 5d69fa3 is a master commit associated with this PR. Results will be available as each benchmark for each run completes. |
Which issue does this PR close?
re #4460
Rationale for this change
make the harness added in #4395 by @mvanschellebeeck easier to find
What changes are included in this PR?
Add a pointer to these tests in the contributor guide
Are these changes tested?
N/A
Are there any user-facing changes?