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