apache / apache/datafusion

Make ExecutionProps a Trait

Aperta
#4,629 0 commenti 0 reazioni 1 assegnatario Rivendicata da @tustvold Vedi su GitHub
enhancement
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.

Ricevi le nuove issue nella tua casella

Un breve riepilogo di issue GitHub adatte ai principianti.