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

ASF GitHub Bot commented on TWILL-216:
--------------------------------------

Github user chtyim commented on a diff in the pull request:

    https://github.com/apache/twill/pull/33#discussion_r101699525
  
    --- Diff: 
twill-core/src/main/java/org/apache/twill/internal/json/TwillRuntimeSpecificationCodec.java
 ---
    @@ -45,6 +45,7 @@
       private static final String TWILL_RUNID = "twillRunId";
       private static final String TWILL_APP_NAME = "twillAppName";
       private static final String RESERVED_MEMORY = "reservedMemory";
    +  private static final String HEAP_TO_RESERVED_RATIO = 
"heapToReservedRatio";
    --- End diff --
    
    for consistency, just have everywhere call it `minHeapRatio`


> Make ratio between total memory and on-heap memory configurable
> ---------------------------------------------------------------
>
>                 Key: TWILL-216
>                 URL: https://issues.apache.org/jira/browse/TWILL-216
>             Project: Apache Twill
>          Issue Type: Improvement
>          Components: yarn
>            Reporter: Yuliya Feldman
>            Assignee: Yuliya Feldman
>
> As of now ratio between on-heap memory and total memory provided to yarn 
> container is hardcoded to 0.7, so if app running in the container needs more 
> reserved memory than on-heap it is not possible to achieve.
> Suggestion is to make it configurable as well as amount of reserved memory



--
This message was sent by Atlassian JIRA
(v6.3.15#6346)

Reply via email to