findepi commented on PR #16573: URL: https://github.com/apache/datafusion/pull/16573#issuecomment-3022365302
It looks we didn't reach a conclusion here yet. There is "just a String" option as currently implemented in this PR. There is "a new struct passed by reference (or Arc)" option as proposed in the comment above. There is also "add SessionConfig by reference (or Arc)" option (https://github.com/apache/datafusion/issues/13519) that I am not very supportive of. @alamb @Omega359 thoughts? -- 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
