Github user vanzin commented on the issue:

    https://github.com/apache/spark/pull/20785
  
    The fix is not correct. If you want to change the order of precedence of 
these configs, you need to change `Utils.getSparkOrYarnConfig`.
    
    Just to double check, this has always worked like this, right? I checked 
both 2.2 and 2.3 and both choose the YARN configuration over the Spark 
configuration when it's set.


---

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

Reply via email to