Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Support Alicloud OSS #681

Draft
wants to merge 9 commits into
base: develop
Choose a base branch
from
Draft

Support Alicloud OSS #681

wants to merge 9 commits into from

Conversation

niejn
Copy link

@niejn niejn commented Jan 3, 2022

Title

Support Alicloud OSS

Motivation

I am currently working for alibaba, and I encounter marvelous smartopen lib when I am coding. However, it doesn't support Alicloud OSS.
So I figured I should add oss support myself.
Contributing to opensource community is always my dream.

Tests

still working on integration tests.

Work in progress

70% finished, would love to hear your comments.

Checklist

Before you create the PR, please make sure you have:

  • Picked a concise, informative and complete title
  • Clearly explained the motivation behind the PR
  • Linked to any existing issues that your PR will be solving
  • Included tests for any new functionality
  • Checked that all unit tests pass

njn007 and others added 7 commits January 2, 2022 10:23
add oss reader
add alicloud oss reader, singlePartReader, multiPartReader
wip: adding oss test code, support oss read and write
@niejn niejn changed the title WIP: Alicloud OSS WIP: Support Alicloud OSS Jan 3, 2022
@mpenkov
Copy link
Collaborator

mpenkov commented Jan 4, 2022

Hi, had a brief look at the changes, looks like you're on the right track. Let me know when the code is ready for a review. Thanks!

@mpenkov mpenkov marked this pull request as draft September 7, 2023 06:47
@mpenkov mpenkov changed the title WIP: Support Alicloud OSS Support Alicloud OSS Sep 7, 2023
@feisan
Copy link

feisan commented Dec 14, 2023

Any updates?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants