apache / apache/datafusion

Substrait physical plan producer emits invalid URIs for LocalFiles

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

Description

The Substrait physical plan producer emits bare paths instead of valid URIs for `LocalFiles` items.

https://github.com/apache/datafusion/blob/1f26716280f330554b37791a057aaff44068698e/datafusion/substrait/src/physical_plan/producer.rs#L65-L67

This produces paths like `path/to/file.parquet` instead of proper URIs like `file:///path/to/file.parquet`.

These bare paths fail `Url::parse()` with "relative URL without a base" since they're not valid URIs per RFC 3986.

Contributor guide

Open the contributing guide

Research direction

Start in datafusion/substrait/src/physical_plan/producer.rs at lines 65-67 and inspect how LocalFiles paths are emitted. Verify the resulting values are valid file URIs that Url::parse() accepts, including the file:///path/to/file.parquet form.

Written by the indexing model from the issue text.

Assessment

Tech stack
rust
Domain
backend
Issue type
Bug
Difficulty
2/5
Estimated time
1-3 hours
Activity status
Stale
Clarity
Clearly specified
Newbie friendliness
55/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.