[ 
https://issues.apache.org/jira/browse/SPARK-2140?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14094244#comment-14094244
 ] 

Chris Cope commented on SPARK-2140:
-----------------------------------

But what about Client.scala L87:
{code}memoryResource.setMemory(args.amMemory + memoryOverhead){code}
Does calculateAMMemory need this logic? I'm sure I'm confusing this more, but 
I'm looking at this ticket and the history of the code(commenting of the code) 
to see what is needed for v1.0.1

> yarn stable client doesn't properly handle MEMORY_OVERHEAD for AM
> -----------------------------------------------------------------
>
>                 Key: SPARK-2140
>                 URL: https://issues.apache.org/jira/browse/SPARK-2140
>             Project: Spark
>          Issue Type: Bug
>          Components: YARN
>    Affects Versions: 1.0.0
>            Reporter: Thomas Graves
>             Fix For: 1.0.1, 1.1.0
>
>
> The yarn stable client doesn't properly remove the MEMORY_OVERHEAD amount 
> from the java heap size, the code to handle that is commented out (see 
> function calculateAMMemory).  We should fix this.



--
This message was sent by Atlassian JIRA
(v6.2#6252)

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

Reply via email to