[GitHub] flink pull request #6297: [FLINK-9777] YARN: JM and TM Memory must be specif...

[GJL]

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

    https://github.com/apache/flink/pull/6297#discussion_r201981340
  
    --- Diff: docs/ops/deployment/yarn_setup.md ---
    @@ -101,12 +101,12 @@ Usage:
        Optional
          -D <arg>                        Dynamic properties
          -d,--detached                   Start detached
    -     -jm,--jobManagerMemory <arg>    Memory for JobManager Container [in 
MB]
    +     -jm,--jobManagerMemory <arg>    Memory for JobManager Container [with 
unit, if not, use MB]
    --- End diff --
    
    nit: I would reword this to: 
    > *Memory for JobManager container with optional unit (default: MB).*


---