alamb commented on PR #26057: URL: https://github.com/apache/datafusion/pull/26057#issuecomment-6467524999
> By implementing the show statements as dedicated execution plans we gain the freedom to adjust those and the informations schema independently. Additionally, you get to still use those statements without needing to make the information schema visible in each catalog (i.e. the dependency on the config setting goes away). Yeah, I agree I do feel like there is this tradeoff between how tightly integrated SHOW TABLES is in datafuson -- however, given the fact that the built in parser handles `SHOW TABLES` etc directly, and that we already have a LogicalPlan variant for one, but not all of the SHOW statements, I am convinced making them full on LogicalPlans is a better path forward than the current status quo While it might be "nicer" in some sense to have fewer LogicalPlan variants, I think this one reasonable -- 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]
