Make ExecutionProps a Trait
- Lingua principale
- Rust
- Stelle
- 9.3k
- Fork
- 2.4k
- Merge medio
- 3g 11h
- PR unite (30g)
- 360
Descrizione
**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**
Guida per i contributori
Apri la guida per i contributori
Valutazione
Questa issue non è ancora stata valutata.