MohamedAbdeen21 commented on PR #11453: URL: https://github.com/apache/datafusion/pull/11453#issuecomment-2229090372
I can't see how that benefits us. Setting an env var just to read it once in a test, which is meant to be a (hopefully) temporary fix, seems ... unnecessary. Plus, other devs WILL run into this issue in their local envs whenever they use the full test command (with the backtrace feature), so hiding the variable in workflow env doesn't seem like a reasonable move. -- 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
