[GitHub] aljoscha commented on issue #7420: [FLINK-11272][flink-yarn] Support for parsing multiple --yarnship par…

2019-01-10 Thread GitBox
aljoscha commented on issue #7420: [FLINK-11272][flink-yarn] Support for parsing multiple --yarnship par… URL: https://github.com/apache/flink/pull/7420#issuecomment-453073668 Thanks again for your contribution! I merged

[GitHub] aljoscha commented on issue #7420: [FLINK-11272][flink-yarn] Support for parsing multiple --yarnship par…

2019-01-10 Thread GitBox
aljoscha commented on issue #7420: [FLINK-11272][flink-yarn] Support for parsing multiple --yarnship par… URL: https://github.com/apache/flink/pull/7420#issuecomment-453037254 Thanks @leesf! I'll merge once Travis is green.

[GitHub] aljoscha commented on issue #7420: [FLINK-11272][flink-yarn] Support for parsing multiple --yarnship par…

2019-01-10 Thread GitBox
aljoscha commented on issue #7420: [FLINK-11272][flink-yarn] Support for parsing multiple --yarnship par… URL: https://github.com/apache/flink/pull/7420#issuecomment-453013004 Could we also have a test for this? Otherwise this looks like a good change, as @walterddr said.