fix(oss): use S3-compatible storage for MinIO#1816
Conversation
Codecov Report❌ Patch coverage is 📢 Thoughts on this report? Let us know! |
|
@chickenlj 你好,能帮我审核一下吗,谢谢您 |
|
Is aws sdk fully compatible with Alibaba Cloud OSS? replace current one or provide a new extension, which do you think is better? |
|
Good point. This PR is not meant to claim that the AWS SDK is a 1:1 replacement for the Alibaba Cloud OSS SDK. The OSS module is now a compatibility layer for S3-compatible object storage, implemented with AWS SDK v2 I kept the existing |
|
This PR updates the OSS-backed distributed storage implementation to use the AWS SDK v2 What changed:
Closes #1795. |
AgentScope-Java Version
2.0.0-SNAPSHOT
Description
This PR updates the OSS extension to use AWS SDK v2 S3-compatible storage, centralizes the object-store helper logic, adds tests, and updates the docs to describe the module as S3-compatible object storage support rather than Alibaba OSS-only support.
The OSS module was using the Alibaba Cloud OSS SDK, which is not compatible with MinIO and other generic S3-compatible stores in this project scenario. This PR switches the storage layer to AWS SDK v2
S3Clientand keeps the existingOss*API surface so downstream code can keep working.How this fixes the issue
S3ObjectStoreSupportto centralize shared S3 object-store logic.OssAgentStateStore,OssBaseStore,OssDistributedStore,OssRemoteSnapshotClient, andOssSnapshotSpecto use the new S3-backed implementation.ossClient(...)as a backwards-compatible builder alias, so existing callers do not need to change immediately..list.jsonentries being picked up as regular filesystem items, and make truncated-list handling null-safe in tests.Verification
mvn -s codex-settings.xml -pl agentscope-extensions/agentscope-extensions-oss test -DskipITsmvn -s codex-settings.xml -pl agentscope-extensions/agentscope-extensions-oss -DskipTests test-compileFixes #1795
Checklist
mvn spotless:applymvn test)