fix(bulkload) follow empiredan advice

apache · Jan 13, 2025 · 4aa4c16 · 4aa4c16
1 parent f34680f
commit 4aa4c16
Showing 1 changed file with 1 addition and 1 deletion.
diff --git a/src/replica/bulk_load/replica_bulk_loader.cpp b/src/replica/bulk_load/replica_bulk_loader.cpp
@@ -596,12 +596,12 @@ void replica_bulk_loader::download_sst_file(
         return;
     }
     // download file succeed, update progress
-    download_file_metas.pop_back();
     update_bulk_load_download_progress(f_size, f_meta.name);
     METRIC_VAR_INCREMENT(bulk_load_download_file_successful_count);
     METRIC_VAR_INCREMENT_BY(bulk_load_download_file_bytes, f_size);
 
     // download next file
+    download_file_metas.pop_back();
     if (!download_file_metas.empty()) {
         _download_files_task[download_file_metas.back().name] = tasking::enqueue(
             LPC_BACKGROUND_BULK_LOAD,