-
Notifications
You must be signed in to change notification settings - Fork 36
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
Feat: Create External Table support for Excel #2652
Conversation
@@ -275,7 +276,8 @@ impl<'a> ExternalDispatcher<'a> { | |||
let tunnel = self.get_tunnel_opts(table.tunnel_id)?; | |||
|
|||
match &table.options { | |||
TableOptions::Internal(TableOptionsInternal { .. }) => unimplemented!(), // Purposely unimplemented. | |||
TableOptions::Internal(TableOptionsInternal { .. }) | |||
| TableOptions::Excel(TableOptionsExcel { .. }) => unimplemented!(), // Purposely unimplemented. |
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.
I would think that we'd be able to implement this, but let's make excel todo!()
rather than unimplemented: we would like to do it eventually, I think.
@@ -887,7 +887,8 @@ impl<'a> SessionPlanner<'a> { | |||
let has_header = storage_options | |||
.inner | |||
.get("has_header") | |||
.map(|val| val.parse::<bool>().unwrap_or_else(|| false))?; | |||
.map(|val| val.parse::<bool>().unwrap_or(false)) |
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.
I just opened a different PR for the table function to have this default to true
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.
Sure, will change it here as well.
@tychoish Can this PR be merged now? |
Feature for #2644
Hello @tychoish
By taking this as reference, I added sheet_name as option in
TableOptionsExcel
- This is different from Bson model.Please review, Thanks.