apache / apache/datafusion-ballista

Provide way to control collect_stat for the table provider

Open
#406 0 comments 1 reaction 0 assignees View on GitHub
enhancement
Dominant language
Rust
Stars
2.1k
Forks
320
Avg merge
1d 22h
Merged PRs (30d)
66

Description

**Is your feature request related to a problem or challenge? Please describe what you are trying to do.**

Now that statistics are extended by @isidentical and others, we should be able to use statistics in Ballista as well.
This has some nice benefits for performance: https://github.com/apache/arrow-datafusion/pull/3889#issuecomment-1283276230

It looks like currently the statistics are disabled in Ballista (as DataFusion was hardcoded to use `collect_stat: false`.

**Describe the solution you'd like**

https://github.com/apache/arrow-datafusion/pull/3846

Use the new setting in DataFusion.

**Describe alternatives you've considered**

**Additional context**

Contributor guide

Open the contributing guide

Research direction

Search the Ballista codebase for the hardcoded `collect_stat: false`, then compare the requested behavior with DataFusion PR #3846. Done means the table provider can use the new DataFusion setting so statistics are available to Ballista.

Written by the indexing model from the issue text.

Assessment

Tech stack
rust
Domain
distributed-systems
Issue type
Feature
Difficulty
3/5
Estimated time
1-2 days
Activity status
Stale
Clarity
Mostly clear
Newbie friendliness
38/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.