[GitHub] flink issue #4622: [FLINK-7294]:[flink-mesos] mesos.resourcemanager.framewor...

2017-09-14 Thread tillrohrmann
Github user tillrohrmann commented on the issue: https://github.com/apache/flink/pull/4622 Thanks for the contribution @bbayani and the review @EronWright. Changes look good to me. Will merge the fix and backport it to the `1.3` release branch. ---

[GitHub] flink issue #4622: [FLINK-7294]:[flink-mesos] mesos.resourcemanager.framewor...

2017-09-13 Thread EronWright
Github user EronWright commented on the issue: https://github.com/apache/flink/pull/4622 @tillrohrmann please take a look and merge if you're OK with it. ---

[GitHub] flink issue #4622: [FLINK-7294]:[flink-mesos] mesos.resourcemanager.framewor...

2017-09-13 Thread bbayani
Github user bbayani commented on the issue: https://github.com/apache/flink/pull/4622 @EronWright Yes, I have verified the case when mesos.resourcemanager.framework.role is not configured in flink-conf.yaml (and thus defaults to *). In this case, it launched all the tasks only on work

[GitHub] flink issue #4622: [FLINK-7294]:[flink-mesos] mesos.resourcemanager.framewor...

2017-09-12 Thread EronWright
Github user EronWright commented on the issue: https://github.com/apache/flink/pull/4622 @bbayani thanks very much for this contribution. Please confirm my understanding of the issue and verification. It appears that the resources that are requested via `TaskInfo` must reference th

[GitHub] flink issue #4622: [FLINK-7294]:[flink-mesos] mesos.resourcemanager.framewor...

2017-09-11 Thread bbayani
Github user bbayani commented on the issue: https://github.com/apache/flink/pull/4622 Can flink-mesos contributors/owners please take a look at this PR? ---