alamb commented on PR #11938: URL: https://github.com/apache/datafusion/pull/11938#issuecomment-2288557502
> @alamb The `[datafusion-cli]` commit was just a follow-up to make the new `FLIGHT_SQL` table "storage" type available in `datafusion-cli` (i.e. register the new config namespace). I'm sorry if that was misleading. The patch is supposed to add support for Arrow Flight (SQL) data sources. FWI I wrote up my thoughts here; https://github.com/apache/datafusion/issues/11979 I hope to review this PR later today One thing that might be useful to ask is "does this code need to be in the datafusion repo, or could it be put in another repo -- like for example https://github.com/datafusion-contrib/datafusion-table-providers 🤔 We could make you your own repo if you wanted -- 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
