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

ASF GitHub Bot commented on FLINK-2641:
---------------------------------------

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

    https://github.com/apache/flink/pull/1129#discussion_r39488877
  
    --- Diff: flink-dist/src/main/flink-bin/bin/taskmanager.sh ---
    @@ -51,13 +51,41 @@ if [[ $STARTSTOP == "start" ]]; then
             fi
         fi
     
    -    if [[ ! ${FLINK_TM_HEAP} =~ ${IS_NUMBER} ]]; then
    -        echo "[ERROR] Configured TaskManager JVM heap size is not a 
number. Please set '$KEY_TASKM_HEAP_MB' in $FLINK_CONF_FILE."
    +    if [[ ! ${FLINK_TM_HEAP} =~ ${IS_NUMBER} ]] || [[ "${FLINK_TM_HEAP}" 
-lt "0" ]]; then
    +        echo "[ERROR] Configured TaskManager JVM heap size is not a 
number. Please set '${KEY_TASKM_MEM_SIZE}' in ${FLINK_CONF_FILE}."
             exit 1
         fi
     
    -    if [ "$FLINK_TM_HEAP" -gt 0 ]; then
    -        export JVM_ARGS="$JVM_ARGS -Xms"$FLINK_TM_HEAP"m 
-Xmx"$FLINK_TM_HEAP"m"
    +    if [ "${FLINK_TM_HEAP}" -gt "0" ]; then
    +
    +        TM_HEAP_SIZE=${FLINK_TM_HEAP}
    +        TM_OFFHEAP_SIZE=0
    +        # some space for Netty initilization
    +        NETTY_BUFFERS=$((1024 * 1024))
    --- End diff --
    
    Are we sure that 1MB is enough for Netty?


> Integrate the off-heap memory configuration with the TaskManager start script
> -----------------------------------------------------------------------------
>
>                 Key: FLINK-2641
>                 URL: https://issues.apache.org/jira/browse/FLINK-2641
>             Project: Flink
>          Issue Type: New Feature
>          Components: Start-Stop Scripts
>    Affects Versions: 0.10
>            Reporter: Stephan Ewen
>            Assignee: Maximilian Michels
>             Fix For: 0.10
>
>
> The TaskManager start script needs to adjust the {{-Xmx}}, {{-Xms}}, and 
> {{-XX:MaxDirectMemorySize}} parameters according to the off-heap memory 
> settings.



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)

Reply via email to