timsaucer commented on PR #1247: URL: https://github.com/apache/datafusion-python/pull/1247#issuecomment-3354008978
This feels like a *lot* of work that is brittle to changes upstream to try to interrupt the error and provide an alternate. It feels like it would be simpler to provide an alternate `SchemaProvider` which would perform like the default in memory schema provider and if the table requested doesn't exist then to do the checks you have for variables in memory that could fit. To me that would be both (1) in line with what I think the real goal here is about searching for tables and (2) far less brittle to changes in the error handling that we're intercepting. What do you think? -- 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: [email protected] For queries about this service, please contact Infrastructure at: [email protected] --------------------------------------------------------------------- To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected]
