vicennial commented on PR #43735: URL: https://github.com/apache/spark/pull/43735#issuecomment-1817850298
@dongjoon-hyun Thank you for the review! I've updated the version and for the deprecated conf `spark.connect.copyFromLocalToFs.allowDestLocal`, I've added it to the `deprecatedSQLConfigs` list so that it would generate a warning if used. Currently, both configs would work but if the new config is set (currently optional), it would override the deprecated config. Eventually, we should remove the deprecated conf and convert the new config from optional to have a default value. -- 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: reviews-unsubscr...@spark.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org --------------------------------------------------------------------- To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org For additional commands, e-mail: reviews-h...@spark.apache.org
