Skip to content

Commit 2b6dc23

Browse files
authored
[#7225]: fix(client-python) minor fix s3a, oss, abs storage handler mapping (#7226)
### What changes were proposed in this pull request? fix [#7225](#7225 (comment)) ### Why are the changes needed? As the issue described, for s3, oss, abs storage handler, the handler map put a class instead of handler instance, which is obviously wrong and can issues like handler method call missing `self` parameter. ### Does this PR introduce _any_ user-facing change? No ### How was this patch tested? I tested on my own machine with remote s3 storage, oss and abs is not tested.
1 parent b79bb52 commit 2b6dc23

File tree

1 file changed

+3
-3
lines changed

1 file changed

+3
-3
lines changed

clients/client-python/gravitino/filesystem/gvfs_storage_handler.py

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -706,9 +706,9 @@ def _get_last_modified(self, entry: Dict):
706706
StorageType.LOCAL: LOCA_HANDLER,
707707
StorageType.HDFS: HDFS_HANDLER,
708708
StorageType.GCS: GCS_HANDLER,
709-
StorageType.S3A: S3StorageHandler,
710-
StorageType.OSS: OSSSecretKeyCredential,
711-
StorageType.ABS: AzureAccountKeyCredential,
709+
StorageType.S3A: S3_HANDLER,
710+
StorageType.OSS: OSS_HANDLER,
711+
StorageType.ABS: ABS_HANDLER,
712712
}
713713

714714

0 commit comments

Comments
 (0)