Renamed a function for clarity. `Master::reconcile(Framework*, const scheduler::Call::Reconcile&)` and `Master::reconcile(Slave*, const vector<ExecutorInfo>&, const vector<Task>& tasks)` are only loosely related. Per discussion on the development list, using overloading to distinguish these two functions is confusing. Hence, rename the latter to `reconcileKnownSlave`.
Review: https://reviews.apache.org/r/52719/ Project: http://git-wip-us.apache.org/repos/asf/mesos/repo Commit: http://git-wip-us.apache.org/repos/asf/mesos/commit/da2ca0f1 Tree: http://git-wip-us.apache.org/repos/asf/mesos/tree/da2ca0f1 Diff: http://git-wip-us.apache.org/repos/asf/mesos/diff/da2ca0f1 Branch: refs/heads/master Commit: da2ca0f17d9ac9a0248edd18ed2cc7c774e985d4 Parents: 97bd957 Author: Neil Conway <neil.con...@gmail.com> Authored: Wed Oct 19 16:32:21 2016 -0700 Committer: Vinod Kone <vinodk...@gmail.com> Committed: Wed Oct 19 16:32:21 2016 -0700 ---------------------------------------------------------------------- src/master/master.cpp | 12 ++++-------- src/master/master.hpp | 7 ++++--- 2 files changed, 8 insertions(+), 11 deletions(-) ---------------------------------------------------------------------- http://git-wip-us.apache.org/repos/asf/mesos/blob/da2ca0f1/src/master/master.cpp ---------------------------------------------------------------------- diff --git a/src/master/master.cpp b/src/master/master.cpp index bf6bb1a..0cdb45f 100644 --- a/src/master/master.cpp +++ b/src/master/master.cpp @@ -5318,10 +5318,9 @@ void Master::reregisterSlave( // Update slave's version after re-registering successfully. slave->version = version; - // Reconcile tasks between master and the slave. - // NOTE: This sends the re-registered message, including tasks - // that need to be reconciled by the slave. - reconcile(slave, executorInfos, tasks); + // Reconcile tasks between master and slave, and send the + // `SlaveReregisteredMessage`. + reconcileKnownSlave(slave, executorInfos, tasks); // If this is a disconnected slave, add it back to the allocator. // This is done after reconciliation to ensure the allocator's @@ -6767,10 +6766,7 @@ void Master::authenticationTimeout(Future<Option<string>> future) } -// NOTE: This function is only called when the slave re-registers -// with a master that already knows about it (i.e., not a failed -// over master). -void Master::reconcile( +void Master::reconcileKnownSlave( Slave* slave, const vector<ExecutorInfo>& executors, const vector<Task>& tasks) http://git-wip-us.apache.org/repos/asf/mesos/blob/da2ca0f1/src/master/master.hpp ---------------------------------------------------------------------- diff --git a/src/master/master.hpp b/src/master/master.hpp index 263ceb4..6d2db9d 100644 --- a/src/master/master.hpp +++ b/src/master/master.hpp @@ -640,9 +640,10 @@ protected: Framework* framework, const std::vector<TaskStatus>& statuses); - // Handles a known re-registering slave by reconciling the master's - // view of the slave's tasks and executors. - void reconcile( + // When a slave that is known to the master re-registers, we need to + // reconcile the master's view of the slave's tasks and executors. + // This function also sends the `ReregisterSlaveMessage`. + void reconcileKnownSlave( Slave* slave, const std::vector<ExecutorInfo>& executors, const std::vector<Task>& tasks);