[GitHub] mesos issue #248: Fix master validation, incorrect detection of dup Executor...

2017-11-03 Thread abudnik
Github user abudnik commented on the issue: https://github.com/apache/mesos/pull/248 If some framework relies on the assumption that there shouldn't be multiple executors running with the same executor_id on the same agent, then this change might lead to bugs. It would be great

[GitHub] mesos issue #248: Fix master validation, incorrect detection of dup Executor...

2017-11-03 Thread abudnik
Github user abudnik commented on the issue: https://github.com/apache/mesos/pull/248 Should we also update comment https://github.com/apache/mesos/blob/master/include/mesos/mesos.proto#L78-L82 ? ---

[GitHub] mesos issue #248: Fix master validation, incorrect detection of dup Executor...

2017-11-03 Thread abudnik
Github user abudnik commented on the issue: https://github.com/apache/mesos/pull/248 We should also update here https://github.com/apache/mesos/blob/master/src/master/master.hpp#L2665-L2667 and maybe in other places, where master/agent checks for duplicate executor ids. ---

[GitHub] mesos pull request #209: Added Andrei Budnik to the contributors list.

2017-05-02 Thread abudnik
GitHub user abudnik opened a pull request: https://github.com/apache/mesos/pull/209 Added Andrei Budnik to the contributors list. You can merge this pull request into a Git repository by running: $ git pull https://github.com/abudnik/mesos contrib Alternatively you can