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,
 }
 
 

Reply via email to