This is an automated email from the ASF dual-hosted git repository.

jshao pushed a commit to branch branch-0.9
in repository https://gitbox.apache.org/repos/asf/gravitino.git


The following commit(s) were added to refs/heads/branch-0.9 by this push:
     new 69b96eec44 [#7225]: fix(client-python) minor fix s3a, oss, abs storage 
handler mapping (#7228)
69b96eec44 is described below

commit 69b96eec4400457c9bb6f37b32afa4579825c39d
Author: github-actions[bot] 
<41898282+github-actions[bot]@users.noreply.github.com>
AuthorDate: Fri May 23 21:05:25 2025 +0800

    [#7225]: fix(client-python) minor fix s3a, oss, abs storage handler mapping 
(#7228)
    
    ### 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.
    
    Co-authored-by: slimtom95 <[email protected]>
---
 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