Make ExecutionProps a Trait
- 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? Please describe what you are trying to do.**
`ExecutionProps` provides the information needed to lower an `Expr` to an `Arc`. Currently it lives on `SessionState` where the additional nesting adds to the confusing config situation (#4349)
**Describe the solution you'd like**
I would like to refactor `ExecutionProps` to be a trait and passed to `create_physical_expr` as `&dyn ExecutionProps`, this could then be implemented by `SessionState` (or `DataFrame` #4626)
This would help reduce the config struct proliferation, whilst keeping the existing separation between the crates.
**Describe alternatives you've considered**
**Additional context**
Contributor guide
Assessment
This issue has not been assessed yet.