isMaster() - still wasn't right

matulef · Aug 18, 2009 · 740318b · 740318b
1 parent 4b4efd2
commit 740318b
Show file tree

Hide file tree

Showing 2 changed files with 12 additions and 1 deletion.
diff --git a/db/repl.h b/db/repl.h
@@ -41,10 +41,18 @@ namespace mongo {
 
     class DBClientConnection;
     class DBClientCursor;
-	// --slave = SimpleSlave
+
+	/* replication slave? (possibly with slave or repl pair nonmaster)
+       --slave cmd line setting -> SimpleSlave
+	*/
 	typedef enum { NotSlave=0, SimpleSlave, ReplPairSlave } SlaveTypes;
 	extern SlaveTypes slave;
+
+	/* true means we are master and doing replication.  if we are not writing to oplog (no --master or repl pairing), 
+	   this won't be true.
+	*/
     extern bool master;
+
     extern int opIdMem;
 
     bool cloneFrom(const char *masterHost, string& errmsg, const string& fromdb, bool logForReplication, 

diff --git a/db/replset.h b/db/replset.h
@@ -111,6 +111,9 @@ namespace mongo {
        If 'client' is not specified, the current client is used.
     */
     inline bool isMaster( const char *client = 0 ) {
+		if( !slave ) 
+			return true;
+
         if ( !client )
             client = database->name.c_str();