Split out integration test plumbing (#2594) (#2300)#2598
Split out integration test plumbing (#2594) (#2300)#2598tustvold merged 3 commits intoapache:masterfrom
Conversation
tustvold
force-pushed
the
collocate-integration-tests
branch
from
426065a to
ea14618
Compare
| let mut file = tempfile::tempfile().unwrap(); | ||
|
|
||
| { | ||
| let file = |
There was a problem hiding this comment.
This is a drive-by cleanup
|
Benchmark runs are scheduled for baseline = 6ab208c and contender = c6e7680. c6e7680 is a master commit associated with this PR. Results will be available as each benchmark for each run completes. |
3 participants
Which issue does this PR close?
Part of #2594
Part of #2300
Rationale for this change
The way the integration test plumbing is muddled with the core logic is incredibly confusing, and complicates splitting apart the crates (#2594) and adding features flags (#2300)
What changes are included in this PR?
This moves the integration test plumbing to the existing integration-testing. I originally moved it to a separate crate, but this seemed redundant
Are there any user-facing changes?
Technically this functionality was being published, it now isn't. I have confirmed DataFusion isn't making use of this.