alamb commented on code in PR #16388:
URL: https://github.com/apache/datafusion/pull/16388#discussion_r2145600999
##########
datafusion/core/src/execution/session_state.rs:
##########
@@ -1675,6 +1675,13 @@ impl ContextProvider for SessionContextProvider<'_> {
.get(name)
.cloned()
.ok_or_else(|| plan_datafusion_err!("table function '{name}' not
found"))?;
+ let dummy_schema = DFSchema::empty();
Review Comment:
I agree long term it would be great to have table functions support
referencing other relations.
However, we would need to update the API more than just this -- right now
TableFunctions have no way of getting inputs (they return TableProviders)
We would have to work out what the API would look like if they got an input
(perhaps they get an input ExecutonPlan)
As you mention, we would also probably have to figure out planning for
LATERALs, etc
--
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
