Skip to content

Commit 9e1a54e

Browse files
rchowellCTTY
authored andcommitted
fix(s3tables): use 's3' as the default scheme (#2313)
## Which issue does this PR close? - Closes #2312 ## What changes are included in this PR? Changes the s3tables default scheme to "s3://" ## Are these changes tested? With this change, I can read files resolved through an s3table catalog now. (cherry picked from commit b06b573)
1 parent 6692e8c commit 9e1a54e

File tree

1 file changed

+1
-1
lines changed

1 file changed

+1
-1
lines changed

crates/catalog/s3tables/src/catalog.rs

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -201,7 +201,7 @@ impl S3TablesCatalog {
201201
// Use provided factory or default to OpenDalStorageFactory::S3
202202
let factory = storage_factory.unwrap_or_else(|| {
203203
Arc::new(OpenDalStorageFactory::S3 {
204-
configured_scheme: "s3a".to_string(),
204+
configured_scheme: "s3".to_string(),
205205
customized_credential_load: None,
206206
})
207207
});

0 commit comments

Comments
 (0)