apache / apache/datafusion

Make ExecutionProps a Trait

Open
#4,629 0 comments 0 reactions 1 assignee Claimed by @tustvold View on GitHub
enhancement
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

Open the contributing guide

Assessment

This issue has not been assessed yet.

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.