ccciudatu commented on PR #11938: URL: https://github.com/apache/datafusion/pull/11938#issuecomment-2286284151
@alamb The PR title is misleading; this is *not* meant to be a `datafusion-cli` patch. It only got renamed because I added a follow-up commit to make the new `FLIGHT_SQL` table "storage" type work with `datafusion-cli` (i.e. register the new config namespace). However, I'm glad this triggered such an interesting high-level discussion about revisiting the cli package. :) I will squash the whole thing in a single commit and restore the original title . Also, I just realized that I need to add proto serialization for the new FlightExec physical plan, so I'll convert this the PR to a draft for now. -- 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
