apache / apache/datafusion

default FileSource::supports_repartitioning to false

Open
#20,096 6 comments 0 reactions 0 assignees View on GitHub
bug
Dominant language
Rust
Stars
9.3k
Forks
2.4k
Avg merge
3d 7h
Merged PRs (30d)
344

Description

### Describe the bug

This seems like an odd thing to default to `true`. I don't know any formats other than csv which would benefit.

I had a regression when upgrading to df52 where my files (which are a custom format) were getting split up randomly and it took a while to realize this new setting was causing it.

### To Reproduce

_No response_

### Expected behavior

_No response_

### Additional context

_No response_

Contributor guide

Open the contributing guide

Research direction

Start by locating FileSource::supports_repartitioning and inspect how its default is defined and consumed. Confirm the behavior for custom file formats and identify the existing tests covering repartitioning. Done means the default is false while formats that need repartitioning can still opt in, with the relevant tests passing.

Written by the indexing model from the issue text.

Assessment

Tech stack
rust
Domain
backend, data-engineering
Issue type
Bug
Difficulty
2/5
Estimated time
1-3 hours
Activity status
Stale
Clarity
Mostly clear
Newbie friendliness
58/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.