-
Notifications
You must be signed in to change notification settings - Fork 753
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
dal2: Implement limited & offset reader for fs and s3 #3985
Conversation
Signed-off-by: Xuanwo <github@xuanwo.io>
Thanks for the contribution! Please review the labels and make any necessary changes. |
Thanks for the contribution! Please review the labels and make any necessary changes. |
@mergify update |
✅ Branch has been successfully updatedHey, I reacted but my real name is @Mergifyio |
This pull request is being automatically deployed with Vercel (learn more). 🔍 Inspect: https://vercel.com/databend/databend/CDHfm4ccMejGAByC5uKZfCxbiuDp |
Codecov Report
@@ Coverage Diff @@
## main #3985 +/- ##
=====================================
Coverage 57% 57%
=====================================
Files 803 803
Lines 42988 43003 +15
=====================================
+ Hits 24548 24561 +13
- Misses 18440 18442 +2
Continue to review full report at Codecov.
|
/lgtm |
Wait for another reviewer approval |
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
Fixed #3937
Changelog
Test Plan
Unit Tests
Stateless Tests