Github user jdef commented on a diff in the pull request: https://github.com/apache/mesos/pull/248#discussion_r148790566 --- Diff: src/master/validation.cpp --- @@ -350,12 +353,13 @@ Option<Error> reregisterSlave( } if (executor.has_executor_id()) { - if (executorIDs.contains(executor.executor_id())) { + IDPair fep = IDPair(executor.framework_id(), executor.executor_id()); --- End diff -- already got feedback that `fep` is probably not a good choice for a name here
---