Extend aggregation fuzz testing with memory-limit tests
- Dominant language
- Rust
- Stars
- 9.3k
- Forks
- 2.4k
- Avg merge
- 3d 7h
- Merged PRs (30d)
- 344
Description
### Is your feature request related to a problem or challenge?
Follow up to https://github.com/apache/datafusion/issues/12114
When implementing https://github.com/apache/datafusion/pull/23657, I have found we could add more test coverages for larger-than-memory execution for ordered aggregation, by extending the aggregation fuzzer.
Existing todo item in the code:
- https://github.com/apache/datafusion/blob/ba8796e460bd60b8239c5f2f51f83d464efc177b/datafusion/core/tests/fuzz_cases/aggregation_fuzzer/context_generator.rs#L46
### Describe the solution you'd like
_No response_
### Describe alternatives you've considered
_No response_
### Additional context
_No response_
Contributor guide
Research direction
Start with the TODO in datafusion/core/tests/fuzz_cases/aggregation_fuzzer/context_generator.rs and inspect the existing aggregation fuzzer cases. Run the current fuzzer tests to understand their setup, then add coverage for larger-than-memory ordered aggregation using memory limits. Done means the new cases exercise and validate that execution path.
Written by the indexing model from the issue text.
Assessment
- Tech stack
- rust
- Domain
- data, testing-qa
- Issue type
- Feature
- Difficulty
- 3/5
- Estimated time
- 1-2 days
- Activity status
- Quiet
- Clarity
- Mostly clear
- Newbie friendliness
- 62/100