haoyangeng-db opened a new pull request, #57730:
URL: https://github.com/apache/spark/pull/57730

   ### What changes were proposed in this pull request?
   
   Completes the Spark 4.0 migration of the `copyFromLocalToFs` 
local-destination gate: `spark.sql.artifact.copyFromLocalToFs.allowDestLocal` 
becomes a static SQL conf and the deprecated 
`spark.connect.copyFromLocalToFs.allowDestLocal` is removed, leaving one conf 
for the gate.
   
   ### Why are the changes needed?
   
   The config controls whether a caller can write to an arbitrary path on the 
driver's local filesystem, so it is meant to be set by whoever starts the 
driver -- as its own doc says, "when starting spark driver". The deprecated 
Connect conf was a static conf and enforced that, but its replacement was 
declared as a runtime conf, so a session could set it and override the value 
the driver was started with. Declaring the replacement in StaticSQLConf 
restores that property.
   
   ### Does this PR introduce _any_ user-facing change?
   
   N/A
   
   ### How was this patch tested?
   
   New and existing tests.
   
   ### Was this patch authored or co-authored using generative AI tooling?
   
   Co-authored by Claude Code


-- 
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]

Reply via email to