This is an automated email from the ASF dual-hosted git repository.
mchades pushed a commit to branch main
in repository https://gitbox.apache.org/repos/asf/gravitino.git
The following commit(s) were added to refs/heads/main by this push:
new 2b6dc23d0e [#7225]: fix(client-python) minor fix s3a, oss, abs storage
handler mapping (#7226)
2b6dc23d0e is described below
commit 2b6dc23d0e3d38336704530f131b33d7bbc0961d
Author: slimtom95 <[email protected]>
AuthorDate: Fri May 23 19:07:35 2025 +0800
[#7225]: fix(client-python) minor fix s3a, oss, abs storage handler mapping
(#7226)
### What changes were proposed in this pull request?
fix
[#7225](https://github.com/apache/gravitino/issues/7225#issue-3085354943)
### 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.
---
clients/client-python/gravitino/filesystem/gvfs_storage_handler.py | 6 +++---
1 file changed, 3 insertions(+), 3 deletions(-)
diff --git a/clients/client-python/gravitino/filesystem/gvfs_storage_handler.py
b/clients/client-python/gravitino/filesystem/gvfs_storage_handler.py
index b1c024a5c2..518e706261 100644
--- a/clients/client-python/gravitino/filesystem/gvfs_storage_handler.py
+++ b/clients/client-python/gravitino/filesystem/gvfs_storage_handler.py
@@ -706,9 +706,9 @@ storage_handlers: Dict[StorageType, StorageHandler] = {
StorageType.LOCAL: LOCA_HANDLER,
StorageType.HDFS: HDFS_HANDLER,
StorageType.GCS: GCS_HANDLER,
- StorageType.S3A: S3StorageHandler,
- StorageType.OSS: OSSSecretKeyCredential,
- StorageType.ABS: AzureAccountKeyCredential,
+ StorageType.S3A: S3_HANDLER,
+ StorageType.OSS: OSS_HANDLER,
+ StorageType.ABS: ABS_HANDLER,
}