flink-issues mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From dawidwys <...@git.apache.org>
Subject [GitHub] flink issue #6297: [FLINK-9777] YARN: JM and TM Memory must be specified wit...
Date Mon, 16 Jul 2018 10:02:38 GMT
Github user dawidwys commented on the issue:

    Changes look good now. I think we are still missing tests for the added behaviour. We
should check:
    - if yarn cli parameters are properly parsed without units
    - if yarn cli parameters are properly parsed with units other than m
    - if the logic for `getJobManagerHeapMemory/getTaskManagerHeapMemory` works properly (if
the old parameter is parsed if the new one is not provided. If the default value is provided
if no option was set.


View raw message