[GitHub] incubator-spark pull request: [SPARK-1090] improvement on spark_sh...

[CodingCat]

Github user CodingCat commented on the pull request:

    https://github.com/apache/incubator-spark/pull/599#issuecomment-35138442
  
    @aarondav  I totally agree that we should deprecate SPARK_MEM, since that 
PR is still in progress (seems dead), I think we should avoid reading this 
variable in SparkContext to set executor memory. Otherwise the user may set 
this variable to limit memory usage of driver but affect the executor memory 
unintentionally  
    
    I created a PR to remove that line 
https://github.com/apache/incubator-spark/pull/602