You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@mesos.apache.org by vi...@apache.org on 2013/03/28 18:33:34 UTC

svn commit: r1462222 - /incubator/mesos/branches/0.12.x/src/detector/detector.cpp

Author: vinodkone
Date: Thu Mar 28 17:33:34 2013
New Revision: 1462222

URL: http://svn.apache.org/r1462222
Log:
Fixed zookeeper detector by ignoring ZNONODE error in detectMaster()
when it happens after getting child znodes but before getting their
data.

Review: https://reviews.apache.org/r/10105

Modified:
    incubator/mesos/branches/0.12.x/src/detector/detector.cpp

Modified: incubator/mesos/branches/0.12.x/src/detector/detector.cpp
URL: http://svn.apache.org/viewvc/incubator/mesos/branches/0.12.x/src/detector/detector.cpp?rev=1462222&r1=1462221&r2=1462222&view=diff
==============================================================================
--- incubator/mesos/branches/0.12.x/src/detector/detector.cpp (original)
+++ incubator/mesos/branches/0.12.x/src/detector/detector.cpp Thu Mar 28 17:33:34 2013
@@ -519,7 +519,7 @@ void ZooKeeperMasterDetectorProcess::det
       //    code (detectMaster()) will be re-tried.
       // 2) If this is due to no masters present (i.e., code == ZNONODE),
       //    updated() will be called and the detectMaster() will be re-tried.
-      if (zk->retryable(code)) {
+      if (zk->retryable(code) || code == ZNONODE) {
         LOG(ERROR) << "Master detector failed to fetch new master pid: "
                    << zk->message(code);
       } else {