Continue to reduce Expr struct size
- 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?
Currently, we already done some work for reducing the Expr size in:
https://github.com/apache/datafusion/pull/16207
We can continue to reduce the Expr struct size
### Describe the solution you'd like
change Expr Alias and OuterReferenceColumn to Box type for reducing expr struct size
### Describe alternatives you've considered
change Expr Alias and OuterReferenceColumn to Box type for reducing expr struct size
### Additional context
_No response_
Contributor guide
Research direction
Review PR #16207 first, then locate the Rust definition and uses of Expr, Alias, and OuterReferenceColumn to trace how these variants are constructed and matched. Run the relevant DataFusion tests after the change; done means the Expr representation is smaller without behavioral regressions.
Written by the indexing model from the issue text.
Assessment
- Tech stack
- rust
- Domain
- backend, performance
- Issue type
- Refactor
- Difficulty
- 4/5
- Estimated time
- 3-5 days
- Activity status
- Stale
- Clarity
- Mostly clear
- Newbie friendliness
- 45/100