You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
{{ message }}
Repository navigation
Failed to plan files: Invalid: Expected a local filesystem path, got a URI in PlanFiles() call in demo_example.cc #341
There is a "file:" prefix getting appended to the avro file path which leads to this error inside arrow localfs.cc (RETURN_NOT_OK(ValidatePath(path));) while running demo_example.cc on WIndows while making the PlanFiles() call.
iceberg-reader.exe "C:\iceberg_warehouse\db" sample "C:\iceberg_warehouse\db\sample\metadata\v5.metadata.json"
Failed to plan files: Invalid: Expected a local filesystem path, got a URI: 'file:/C:/iceberg_warehouse/db/sample/metadata/snap-4130091257426129961-1-0b7261a2-d123-4635-82a6-0a5a30e19a06.avro'
Could you please check if C:\iceberg_warehouse\db\sample\metadata\v5.metadata.json has used file:/C:/iceberg_warehouse/db/sample/metadata/snap-4130091257426129961-1-0b7261a2-d123-4635-82a6-0a5a30e19a06.avro as the manifest list file location? I suspect that the Arrow local filesystem implementation does not handle file:/ prefix well.
I would like to work on this. Plan: correct the file-path handling in PlanFiles() so local/Windows paths are not mis-parsed as URIs, and add a regression test covering the Windows path case. Will disclose AI-assisted authoring in the PR per the Iceberg AI-contribution guidelines.
Activity
wgtmac commentedon Nov 25, 2025
Could you please check if
C:\iceberg_warehouse\db\sample\metadata\v5.metadata.jsonhas usedfile:/C:/iceberg_warehouse/db/sample/metadata/snap-4130091257426129961-1-0b7261a2-d123-4635-82a6-0a5a30e19a06.avroas the manifest list file location? I suspect that the Arrow local filesystem implementation does not handlefile:/prefix well.yadavay-amzn commentedon Jul 6, 2026
I would like to work on this. Plan: correct the file-path handling in PlanFiles() so local/Windows paths are not mis-parsed as URIs, and add a regression test covering the Windows path case. Will disclose AI-assisted authoring in the PR per the Iceberg AI-contribution guidelines.
Fix URI-vs-path handling in PlanFiles() for local/Windows paths (apac…
Make file-URI tests cross-platform (fix Windows CI) (apache#341)
Extract shared IsUriScheme helper; use std::ranges::replace (apache#341)
Clarify colon_pos check in IsUriScheme (apache#341)
Reuse scheme colon position via IsUriScheme out-param (apache#341)