AngersZhuuuu opened a new pull request #31598:
URL: https://github.com/apache/spark/pull/31598


   
   ### What changes were proposed in this pull request?
   Some user who not so clear about spark when build SparkSession
   ```
   SparkSession.builder().config()
   ```
   In this method user may config `spark.driver.memory` then he will think when 
submit job, spark driver's memory is change. But when we run this code, JVM has 
been  started,  so this configuration won't work at all.  And in Spark UI, it 
will show `spark.driver.memory` as this configuration set by 
`SparkSession.builder().config()`. This makes user and administer confuse.
   So we should ignore such as wrong way and logwarn.
   
   
   ### Why are the changes needed?
   Make usage more clear
   
   
   ### Does this PR introduce _any_ user-facing change?
   No, this pr only handle wrong use case.
   
   ### How was this patch tested?
   WIP
   


----------------------------------------------------------------
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:
us...@infra.apache.org



---------------------------------------------------------------------
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h...@spark.apache.org

Reply via email to