save

Author: xy720

be/src/agent/task_worker_pool.cpp

@@ -1026,7 +1026,6 @@ void update_tablet_meta_callback(StorageEngine& engine, const TAgentTaskRequest&
                 rowset_meta->tablet_schema()->set_skip_write_index_on_load(
                         tablet_meta_info.skip_write_index_on_load);
             }
-            tablet->tablet_schema_unlocked()->set_skip_write_index_on_load(
                     tablet_meta_info.skip_write_index_on_load);
             need_to_save = true;
         }
@@ -1277,10 +1276,9 @@ void report_tablet_callback(CloudStorageEngine& engine, const ClusterInfo* clust
                          "WorkPoolCloudReportTablet.report_tablet_callback.skip"))) {
         LOG(WARNING) << "report version " << report_version << " change to " << s_report_version;
         DorisMetrics::instance()->report_all_tablets_requests_skip->increment(1);
+        int64_t expected = 0;
         int64_t current_time = time(nullptr);
-        if (s_tablet_report_failure_start_time.load() == 0) {
-            s_tablet_report_failure_start_time.store(current_time);
-        }
+        s_tablet_report_failure_start_time.compare_exchange_strong(expected, current_time);
         DorisMetrics::instance()->tablet_report_continuous_failure_duration_s->set_value(
                 current_time - s_tablet_report_failure_start_time);
         return;
@@ -1293,10 +1291,9 @@ void report_tablet_callback(CloudStorageEngine& engine, const ClusterInfo* clust
     report_tablet_total << 1;
     if (!succ) [[unlikely]] {
         report_tablet_failed << 1;
+        int64_t expected = 0;
         int64_t current_time = time(nullptr);
-        if (s_tablet_report_failure_start_time.load() == 0) {
-            s_tablet_report_failure_start_time.store(current_time);
-        }
+        s_tablet_report_failure_start_time.compare_exchange_strong(expected, current_time);
         DorisMetrics::instance()->tablet_report_continuous_failure_duration_s->set_value(
                 current_time - s_tablet_report_failure_start_time);
     } else {