apache / apache/datafusion

[Discussion] Add async call to TableFunctionImpl trait

Open
#17,663 1 comment 3 reactions 0 assignees View on GitHub
Dominant language
Rust
Stars
9.3k
Forks
2.4k
Avg merge
3d 7h
Merged PRs (30d)
344

Description

Currently, there is only a sync call method in the trait

```rust
/// A trait for table function implementations
pub trait TableFunctionImpl: Debug + Sync + Send {
/// Create a table provider
fn call(&self, args: &[Expr]) -> Result>;
}
```

It would be helpful if we add an async call method to the trait.

Contributor guide

Open the contributing guide

Research direction

Start by locating the TableFunctionImpl trait and reviewing its existing implementations and call sites. Determine the async API shape and update the relevant coverage so the trait supports asynchronous table-function calls without losing the existing synchronous behavior; done means the design is agreed and the affected tests pass.

Written by the indexing model from the issue text.

Assessment

Tech stack
rust
Domain
databases
Issue type
Feature
Difficulty
4/5
Estimated time
3-5 days
Activity status
Stale
Clarity
Mostly clear
Newbie friendliness
35/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.