Skip to content
New issue

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

#

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

Already on GitHub? # to your account

Jump to bottom

opendal: Bump to v0.2.2 #4433

Merged
merged 3 commits into from
Mar 14, 2022
Merged

opendal: Bump to v0.2.2 #4433

merged 3 commits into from
Mar 14, 2022

Conversation

Xuanwo
Copy link
Member

@Xuanwo Xuanwo commented Mar 14, 2022 •
edited
Loading

Signed-off-by: Xuanwo github@xuanwo.io

I hereby agree to the terms of the CLA available at: https://databend.rs/dev/policies/cla/

Summary

Bump opendal to v0.2.2.

Databend related changes:

  • Fix stat root path doesn't return dir correctly
  • Fix random EOF error on fs backend
  • Enable logging and matrics for opendal

Changelog

  • New Feature
  • Bug Fix

Test Plan

Unit Tests

Stateless Tests

@Xuanwo
opendal: Bump to v0.2.2
0ee247f 
- Fix random ROF error on fs backend
- Enable logging and matrics for opendal

Signed-off-by: Xuanwo <github@xuanwo.io>
@Xuanwo Xuanwo requested a review from BohuTANG as a code owner March 14, 2022 04:43
@vercel
Copy link

vercel bot commented Mar 14, 2022 •
edited
Loading

This pull request is being automatically deployed with Vercel (learn more).
To see the status of your deployment, click below or on the icon next to each commit.

🔍 Inspect: https://vercel.com/databend/databend/3EjFNiqVYqNzoFDX9d1qJDc35U9Y
✅ Preview: https://databend-git-fork-xuanwo-opendal-v022-databend.vercel.app

@Xuanwo Xuanwo requested review from sundy-li and dantengsky March 14, 2022 04:43
@mergify
Copy link
Contributor

mergify bot commented Mar 14, 2022

Thanks for the contribution!
I have applied any labels matching special text in your PR Changelog.

Please review the labels and make any necessary changes.

@mergify mergify bot added pr-feature this PR introduces a new feature to the codebase pr-bugfix this PR patches a bug in codebase labels Mar 14, 2022
@Xuanwo
Make taplo happy
05b7342 
Signed-off-by: Xuanwo <github@xuanwo.io>
@dantengsky
Copy link
Member

/LGTM

@sundy-li
Copy link
Member

@Xuanwo Need to export BoxedObjectStream then Layer can override the list method.

@Xuanwo
Copy link
Member Author

Xuanwo commented Mar 14, 2022 •
edited
Loading

@Xuanwo Need to export BoxedObjectStream then Layer can override the list method.

Got it, I will fix it in apache/opendal#147

@Xuanwo Xuanwo mentioned this pull request Mar 14, 2022
Closed
@mergify mergify bot merged commit 9cbed79 into databendlabs:main Mar 14, 2022
@Xuanwo Xuanwo deleted the opendal-v0.2.2 branch March 14, 2022 06:38
# for free to join this conversation on GitHub. Already have an account? # to comment
Reviewers

@sundy-li sundy-li sundy-li approved these changes

@databend-bot databend-bot databend-bot approved these changes

@BohuTANG BohuTANG Awaiting requested review from BohuTANG

@dantengsky dantengsky Awaiting requested review from dantengsky

Assignees
No one assigned
Labels
need-review pr-bugfix this PR patches a bug in codebase pr-feature this PR introduces a new feature to the codebase
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
4 participants
@Xuanwo @dantengsky @sundy-li @databend-bot