Fix snapshot's making header bug (#362)

apache · Nov 28, 2018 · aa27d0e · aa27d0e
1 parent 1ffc294
commit aa27d0e
Showing 1 changed file with 0 additions and 11 deletions.
diff --git a/be/src/olap/olap_snapshot.cpp b/be/src/olap/olap_snapshot.cpp
@@ -377,15 +377,6 @@ OLAPStatus OLAPEngine::_create_snapshot_files(
         header_locked = false;
         _update_header_file_info(shortest_versions, new_olap_header);
 
-        // save new header
-        if ((res = OlapHeaderManager::save(store, ref_olap_table->tablet_id(),
-                ref_olap_table->schema_hash(), new_olap_header)) != OLAP_SUCCESS) {
-            LOG(WARNING) << "save header error. [table=" << ref_olap_table->full_name()
-                    << "tablet_id=" << ref_olap_table->full_name()
-                    << ", schema_hash=" << ref_olap_table->schema_hash();
-             break;
-         }
-
         // save new header to snapshot header path
         res = new_olap_header->save(header_path);
         if (res != OLAP_SUCCESS) {
@@ -791,8 +782,6 @@ OLAPStatus OLAPEngine::_generate_new_header(
     _update_header_file_info(version_entity_vec, new_olap_header);
     new_olap_header->set_shard(new_shard);
 
-    _update_header_file_info(version_entity_vec, new_olap_header);
-
     res = OlapHeaderManager::save(store, tablet->tablet_id(), tablet->schema_hash(), new_olap_header);
     if (res != OLAP_SUCCESS) {
         OLAP_LOG_WARNING("fail to save olap header to new db. [res=%d]", res);