You are viewing a plain text version of this content. The canonical link for it is here.
Posted to github@datafusion.apache.org by "Yuvraj-cyborg (via GitHub)" <gi...@apache.org> on 2025/12/09 15:27:35 UTC
[PR] Feat: Add an option for fast tests by gating slow tests to extended_tests feature [datafusion]
Yuvraj-cyborg opened a new pull request, #19237:
URL: https://github.com/apache/datafusion/pull/19237
## Which issue does this PR close?
Closes #19228
## Rationale for this change
This PR gates all fuzz tests behind the ```extended_tests``` feature flag, allowing developers to run a fast test suite (ideally under 1 minute) during development while still being able to run the full test suite in CI or when needed.
## What changes are included in this PR?
Added #[cfg(feature = "extended_tests")] to all fuzz test modules in mod.rs. This includes the following modules:
- aggregate_fuzz
- distinct_count_string_fuzz
- join_fuzz
- merge_fuzz
- sort_fuzz
- sort_query_fuzz
- topk_filter_pushdown
- aggregation_fuzzer
- equivalence
- pruning
- limit_fuzz
- sort_preserving_repartition_fuzz
- window_fuzz
- Utility modules
## Are these changes tested?
Without the extended_tests feature:
```cargo test --package datafusion --test fuzz -- --list```
This shows 0 tests.
With the extended_tests feature:
```cargo test --package datafusion --test fuzz --features extended_tests -- --list```
This shows all ~100+ fuzz tests.
## Are there any user-facing changes?
Yes. Users can now run:
Fast test suite (default):
```cargo test```
or
```cargo nextest run```
Full test suite with fuzz tests:
```cargo test --features extended_tests```
or
```cargo nextest run --features extended_tests```
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: github-unsubscribe@datafusion.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
---------------------------------------------------------------------
To unsubscribe, e-mail: github-unsubscribe@datafusion.apache.org
For additional commands, e-mail: github-help@datafusion.apache.org
Re: [PR] Feat: Add an option for fast tests by gating slow tests to extended_tests feature [datafusion]
Posted by "alamb (via GitHub)" <gi...@apache.org>.
alamb commented on PR #19237:
URL: https://github.com/apache/datafusion/pull/19237#issuecomment-3641874757
🎉
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: github-unsubscribe@datafusion.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
---------------------------------------------------------------------
To unsubscribe, e-mail: github-unsubscribe@datafusion.apache.org
For additional commands, e-mail: github-help@datafusion.apache.org
Re: [PR] Feat: Add an option for fast tests by gating slow tests to extended_tests feature [datafusion]
Posted by "Yuvraj-cyborg (via GitHub)" <gi...@apache.org>.
Yuvraj-cyborg commented on PR #19237:
URL: https://github.com/apache/datafusion/pull/19237#issuecomment-3635535390
@kosiew Thanks I changed it, looks good now ??
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: github-unsubscribe@datafusion.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
---------------------------------------------------------------------
To unsubscribe, e-mail: github-unsubscribe@datafusion.apache.org
For additional commands, e-mail: github-help@datafusion.apache.org
Re: [PR] Feat: Add an option for fast tests by gating slow tests to extended_tests feature [datafusion]
Posted by "Yuvraj-cyborg (via GitHub)" <gi...@apache.org>.
Yuvraj-cyborg commented on PR #19237:
URL: https://github.com/apache/datafusion/pull/19237#issuecomment-3635784309
@2010YOUY01 @kosiew Can we have it merged !?
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: github-unsubscribe@datafusion.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
---------------------------------------------------------------------
To unsubscribe, e-mail: github-unsubscribe@datafusion.apache.org
For additional commands, e-mail: github-help@datafusion.apache.org
Re: [PR] Feat: Add an option for fast tests by gating slow tests to extended_tests feature [datafusion]
Posted by "2010YOUY01 (via GitHub)" <gi...@apache.org>.
2010YOUY01 merged PR #19237:
URL: https://github.com/apache/datafusion/pull/19237
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: github-unsubscribe@datafusion.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
---------------------------------------------------------------------
To unsubscribe, e-mail: github-unsubscribe@datafusion.apache.org
For additional commands, e-mail: github-help@datafusion.apache.org
Re: [PR] Feat: Add an option for fast tests by gating slow tests to extended_tests feature [datafusion]
Posted by "2010YOUY01 (via GitHub)" <gi...@apache.org>.
2010YOUY01 commented on PR #19237:
URL: https://github.com/apache/datafusion/pull/19237#issuecomment-3635854651
> @2010YOUY01 @kosiew Can we have it merged !?
We typically wait 24 hours between approval and merge (except for trivial PRs like typo fixes) to give others a chance to review.
Here is the reference: https://datafusion.apache.org/contributor-guide/index.html#pull-request-overview
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: github-unsubscribe@datafusion.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
---------------------------------------------------------------------
To unsubscribe, e-mail: github-unsubscribe@datafusion.apache.org
For additional commands, e-mail: github-help@datafusion.apache.org
Re: [PR] Feat: Add an option for fast tests by gating slow tests to extended_tests feature [datafusion]
Posted by "Yuvraj-cyborg (via GitHub)" <gi...@apache.org>.
Yuvraj-cyborg commented on PR #19237:
URL: https://github.com/apache/datafusion/pull/19237#issuecomment-3632865930
@2010YOUY01 hey , does this seem good ?
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: github-unsubscribe@datafusion.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
---------------------------------------------------------------------
To unsubscribe, e-mail: github-unsubscribe@datafusion.apache.org
For additional commands, e-mail: github-help@datafusion.apache.org