2010YOUY01 commented on PR #7058: URL: https://github.com/apache/arrow-datafusion/pull/7058#issuecomment-1652352286
> Looking better @2010YOUY01 -- thank you > > What do you think about making this code only apply in `datafusion-cli` rather than the core? Thanks for the review again! @alamb Is this because there might be some other unexpected behavior we might not be aware of for now? Since path expansion might also be useful if someone wants to write some simple testing program using DataFusion as library: How about adding a Catalog configuration option like `use_path_shell_expansion`, and turn it off in default config (also suggest not to use it in production in doc), and turn it on in CLI by default? -- 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]
