milenkovicm commented on code in PR #14631:
URL: https://github.com/apache/datafusion/pull/14631#discussion_r1956211885
##########
datafusion/sql/src/statement.rs:
##########
@@ -1709,18 +1709,22 @@ impl<S: ContextProvider> SqlToRel<'_, S> {
// Do a table lookup to verify the table exists
let table_ref =
self.object_name_to_table_reference(table_name.clone())?;
let table_source =
self.context_provider.get_table_source(table_ref.clone())?;
- let schema = (*table_source.schema()).clone();
- let schema = DFSchema::try_from(schema)?;
- let scan =
- LogicalPlanBuilder::scan(table_ref.clone(), table_source,
None)?.build()?;
+ let schema = table_source.schema().to_dfschema_ref()?;
+ let scan = LogicalPlanBuilder::scan(
+ //object_name_to_string(&table_name),
Review Comment:
it showed up after rebase, so i left it in case i need to go back to i. will
be removed
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: github-unsubscr...@datafusion.apache.org
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org
---------------------------------------------------------------------
To unsubscribe, e-mail: github-unsubscr...@datafusion.apache.org
For additional commands, e-mail: github-h...@datafusion.apache.org
