apache / apache/datafusion

Make TaskContext wrap SessionState

Open
#10,631 3 comments 0 reactions 0 assignees 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?

I have a user defined table function `FlattenTableFunc` whice needs reading data from other table provider. To serialize execution plan `FlattenExec`, I can't put `SessionState` in it directly. So I have to reconstruct `SessionState` in `FlattenExec::execute` method.

### Describe the solution you'd like

I see `TaskContext` is just part of `SessionState`. Could we just make `TaskContext` wrap `SessionState`?

### Describe alternatives you've considered

_No response_

### Additional context

_No response_

Contributor guide

Open the contributing guide

Research direction

Start by reading the TaskContext and SessionState definitions, then inspect the FlattenExec::execute entry point described in the issue. Determine how execution-plan serialization and access to table providers should work when TaskContext wraps SessionState; done means the execution path no longer reconstructs SessionState in FlattenExec::execute.

Written by the indexing model from the issue text.

Assessment

Tech stack
rust
Domain
backend
Issue type
Feature
Difficulty
5/5
Estimated time
Over a week
Activity status
Stale
Clarity
Mostly clear
Newbie friendliness
32/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.