linhongliu-db commented on pull request #30363: URL: https://github.com/apache/spark/pull/30363#issuecomment-770925696
@srowen, Thanks for reviewing. I think this PR is a kind of bug fix since the `set -v` doesn't output expected results in some cases. Also, even though it would work later after the SQLConf objects are used, it still may mislead users if the outputs of `set -v` are different when run multiple times. Regarding the complexity, actually, this PR almost has no side effect on existing code. even though there are exceptions during eagerly loading, it will be catched and won't break the spark ---------------------------------------------------------------- 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. For queries about this service, please contact Infrastructure at: [email protected] --------------------------------------------------------------------- To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected]
