Browse Source

[1068] made sure Updater::committed_ is updated immediately after commit.

JINMEI Tatuya 13 years ago
parent
commit
4ef59f25a4
1 changed files with 1 additions and 2 deletions
  1. 1 2
      src/lib/datasrc/database.cc

+ 1 - 2
src/lib/datasrc/database.cc

@@ -750,13 +750,12 @@ DatabaseUpdater::commit() {
                   << db_name_);
     }
     accessor_->commitUpdateZone();
+    committed_ = true; // make sure the destructor won't trigger rollback
 
     // We release the accessor immediately after commit is completed so that
     // we don't hold the possible internal resource any longer.
     accessor_.reset();
 
-    committed_ = true;
-
     logger.debug(DBG_TRACE_DATA, DATASRC_DATABASE_UPDATER_COMMIT)
         .arg(zone_name_).arg(zone_class_).arg(db_name_);
 }