TisonKun commented on issue #6491: [FLINK-10057] Update FlinkYarnSessionCli.java
URL: https://github.com/apache/flink/pull/6491#issuecomment-430986516
Thanks for the pick-up @GJL !
This is an automated message from the Apache
TisonKun commented on issue #6491: [FLINK-10057] Update FlinkYarnSessionCli.java
URL: https://github.com/apache/flink/pull/6491#issuecomment-425415727
@uce I understand your concerned and sorry to involve you here.
This is an
TisonKun commented on issue #6491: [FLINK-10057] Update FlinkYarnSessionCli.java
URL: https://github.com/apache/flink/pull/6491#issuecomment-425333772
@lzqdename From my side this pull request goes the right direction and now
it is ready to be merged.
However, I am not a committer
TisonKun commented on issue #6491: [FLINK-10057] Update FlinkYarnSessionCli.java
URL: https://github.com/apache/flink/pull/6491#issuecomment-410494126
cc @twalthr @StephanEwen
This is an automated message from the Apache Git
TisonKun commented on issue #6491: [FLINK-10057] Update FlinkYarnSessionCli.java
URL: https://github.com/apache/flink/pull/6491#issuecomment-410489497
@lzqdename thanks for your PR!
this change makes sense. you could pay attention to the code style by
looking at code around, and it