Skip to content

Make ExecutionProps a Trait #4629

@tustvold

Description

@tustvold

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<dyn PhysicalExpr>. 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

Metadata

Metadata

Assignees

Labels

enhancementNew feature or request

Type

No type

Projects

No projects

Milestone

No milestone

Relationships

None yet

Development

No branches or pull requests

Issue actions