FastRestore:Remove dbprintf

fdbserver/RestoreMaster.actor.cpp

@@ -1223,7 +1223,7 @@ ACTOR static Future<Void> finishRestore(Reference<RestoreMasterData> self, Datab
 				//std::vector<RestoreCommonReply> reps =  wait( getAll(cmdReplies) );
 				cmdReplies.clear();
 			}
-			dbprintf("All restore workers have quited\n");
+			printf("All restore workers have quited\n");
 
 			break;
 		} catch(Error &e) {

fdbserver/RestoreUtil.h

@@ -34,13 +34,6 @@
 #include <cstdarg>
 
 
-inline void dbprintf(const char* fmt) {
-	printf(fmt);
-}
-
-inline void dbprintf(const char* fmt, va_list va) {
-	printf(fmt, va);
-}
 
 
 // TODO: To remove unused command enum. and re-order the command sequence

fdbserver/TLogServer.actor.cpp

@@ -1596,6 +1596,8 @@ ACTOR Future<Void> tLogCommit(
 		return Void();
 	}
 
+	// The logic of increasing logData->version must be atomic in a process, i.e, not including wait() or yield.
+	// Otherwise, the duplicate req (with the same preVersion) can be executed twice
 	if (logData->version.get() == req.prevVersion) {  // Not a duplicate (check relies on no waiting between here and self->version.set() below!)
 		if(req.debugID.present())
 			g_traceBatch.addEvent("CommitDebug", tlogDebugID.get().first(), "TLog.tLogCommit.Before");