Skip to content

Commit c7fc8a5

Browse files
authored
Merge pull request #68 from RelationalAI/gb/fix-routing-factory
Fix routing factory to consider file paths
2 parents 2b1d21b + eb2517b commit c7fc8a5

1 file changed

Lines changed: 1 addition & 8 deletions

File tree

  • crates/iceberg/src/io/storage/opendal

crates/iceberg/src/io/storage/opendal/mod.rs

Lines changed: 1 addition & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -587,14 +587,7 @@ impl StorageFactory for OpenDalRoutingStorageFactory {
587587
})?;
588588
let scheme = Url::parse(&location)
589589
.map(|u| u.scheme().to_string())
590-
.map_err(|e| {
591-
Error::new(
592-
ErrorKind::DataInvalid,
593-
format!(
594-
"OpenDalRoutingStorageFactory: failed to parse metadata location URL: {e}"
595-
),
596-
)
597-
})?;
590+
.unwrap_or_else(|_| "file".to_string());
598591

599592
// Strip internal keys so they don't leak into the OpenDAL operator config.
600593
props.remove(PROP_TABLE_IDENT);

0 commit comments

Comments
 (0)