-
Notifications
You must be signed in to change notification settings - Fork 530
feat: add url support for blob #5766
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
base: main
Are you sure you want to change the base?
Conversation
845a2a5 to
738e371
Compare
885162f to
d01ec72
Compare
d01ec72 to
80d3408
Compare
Codecov Report❌ Patch coverage is 📢 Thoughts on this report? Let us know! |
80d3408 to
ecdf865
Compare
Wondering if we surely introduce this URL-based approach to take a row or rows, should we limit this to fields that use Blob API? |
I think we should only allow blob api to get URLs, which is the BlobFile. |
|
opendal has a native http service support, maybe we can just use that |
ecdf865 to
b178a9c
Compare
The unit test test_blob_https_url_and_range_s3_roundtrip shows the case, it could be ran as below.