fsspec / ossfs

fsspec filesystem for Alibaba Cloud (Aliyun) Object Storage System (OSS)
Apache License 2.0
19 stars 10 forks source link

Implement `remove` related operations in `AioOSSFileSystem` #109

Closed karajan1001 closed 1 year ago

karajan1001 commented 1 year ago

fix: #93

  1. Add rm, rm_file to AioOSSFileSystem
  2. Add related tests for rm operations.
  3. Implement checksum and ukey for AioOSSFileSystem
codecov-commenter commented 1 year ago

Codecov Report

Patch coverage: 83.33% and project coverage change: -0.39 :warning:

Comparison is base (971d9b4) 92.24% compared to head (ecae7ec) 91.85%.

:mega: This organization is not using Codecov’s GitHub App Integration. We recommend you install it so Codecov can continue to function properly for your repositories. Learn more

Additional details and impacted files ```diff @@ Coverage Diff @@ ## main #109 +/- ## ========================================== - Coverage 92.24% 91.85% -0.39% ========================================== Files 13 13 Lines 1548 1597 +49 Branches 288 299 +11 ========================================== + Hits 1428 1467 +39 - Misses 81 88 +7 - Partials 39 42 +3 ``` | [Impacted Files](https://codecov.io/gh/fsspec/ossfs/pull/109?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=None) | Coverage Δ | | |---|---|---| | [src/ossfs/base.py](https://codecov.io/gh/fsspec/ossfs/pull/109?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=None#diff-c3JjL29zc2ZzL2Jhc2UucHk=) | `86.95% <75.00%> (-2.66%)` | :arrow_down: | | [src/ossfs/core.py](https://codecov.io/gh/fsspec/ossfs/pull/109?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=None#diff-c3JjL29zc2ZzL2NvcmUucHk=) | `82.81% <75.00%> (-0.66%)` | :arrow_down: | | [src/ossfs/async\_oss.py](https://codecov.io/gh/fsspec/ossfs/pull/109?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=None#diff-c3JjL29zc2ZzL2FzeW5jX29zcy5weQ==) | `78.20% <85.36%> (+1.24%)` | :arrow_up: | | [src/ossfs/exceptions.py](https://codecov.io/gh/fsspec/ossfs/pull/109?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=None#diff-c3JjL29zc2ZzL2V4Y2VwdGlvbnMucHk=) | `62.50% <100.00%> (-2.21%)` | :arrow_down: | | [tests/test\_object.py](https://codecov.io/gh/fsspec/ossfs/pull/109?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=None#diff-dGVzdHMvdGVzdF9vYmplY3QucHk=) | `99.49% <100.00%> (+<0.01%)` | :arrow_up: | Help us with your feedback. Take ten seconds to tell us [how you rate us](https://about.codecov.io/nps?utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=None). Have a feature suggestion? [Share it here.](https://app.codecov.io/gh/feedback/?utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=None)

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Do you have feedback about the report comment? Let us know in this issue.