-
Notifications
You must be signed in to change notification settings - Fork 517
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
docs(integration/object_store): add example for datafusion #5543
Conversation
Hi, this example is excellent. Could we include it in the README to make it more visible? |
Ok, I'll add it later. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Love it -- thank you @meteorgan (and @Xuanwo for the pointer)
let op = Operator::new(builder) | ||
.map_err(|err| DataFusionError::External(Box::new(err)))? | ||
.finish(); | ||
let store = object_store_opendal::OpendalStore::new(op); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
❤️ this is very cool
43704b5
to
45e0dde
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Really nice, thank you @meteorgan
Which issue does this PR close?
Part of: #4599
Rationale for this change
What changes are included in this PR?
add an example to demonstrate querying data in a S3 bucket using DataFusion and
object_store_opendal
Are there any user-facing changes?