gaogaotiantian commented on PR #53306: URL: https://github.com/apache/spark/pull/53306#issuecomment-3633685313
> Sure - let me rebase... (although conflict is because of https://github.com/apache/spark/pull/53388 - not sure why duplicate?) That would be my fault. I did not realize this PR exists when I fix the `worker`. Whether we should have an envvar fallback is a decision for @HyukjinKwon . Personally I think it's okay to just replace the old mechanism. We would need a whole config path for the fallback to work. With the heavily executed CI I think we can validate this local scope change pretty well. -- 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] --------------------------------------------------------------------- To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected]
