code format

Former-commit-id: c97b162cf28301a5b1838377bdb71d4ad14daca2

cpp/src/db/DBImpl.cpp

@@ -898,34 +898,33 @@ DBImpl::BackgroundBuildIndex() {
     meta_ptr_->FilesToIndex(to_index_files);
     Status status;
 
-//    scheduler::BuildIndexJobPtr
-//        job = std::make_shared<scheduler::BuildIndexJob>(0, meta_ptr_, options_);
+    scheduler::BuildIndexJobPtr
+        job = std::make_shared<scheduler::BuildIndexJob>(0, meta_ptr_, options_);
 
     // step 2: put build index task to scheduler
-//    for (auto &file : to_index_files) {
-//        scheduler::TableFileSchemaPtr file_ptr = std::make_shared<meta::TableFileSchema>(file);
-//        job->AddToIndexFiles(file_ptr);
-//    }
-//    scheduler::JobMgrInst::GetInstance()->Put(job);
-//    job->WaitBuildIndexFinish();
-//    if (!job->GetStatus().ok()) {
-//        Status status = job->GetStatus();
-//        ENGINE_LOG_ERROR << "Building index failed: " << status.ToString();
-//    }
-
     for (auto &file : to_index_files) {
-        std::cout << "get to index file" << std::endl;
-        status = BuildIndex(file);
-        if (!status.ok()) {
-            ENGINE_LOG_ERROR << "Building index for " << file.id_ << " failed: " << status.ToString();
-        }
-
-        if (shutting_down_.load(std::memory_order_acquire)) {
-            ENGINE_LOG_DEBUG << "Server will shutdown, skip build index action";
-            break;
-        }
+        scheduler::TableFileSchemaPtr file_ptr = std::make_shared<meta::TableFileSchema>(file);
+        job->AddToIndexFiles(file_ptr);
+    }
+    scheduler::JobMgrInst::GetInstance()->Put(job);
+    job->WaitBuildIndexFinish();
+    if (!job->GetStatus().ok()) {
+        Status status = job->GetStatus();
+        ENGINE_LOG_ERROR << "Building index failed: " << status.ToString();
     }
 
+//    for (auto &file : to_index_files) {
+//        status = BuildIndex(file);
+//        if (!status.ok()) {
+//            ENGINE_LOG_ERROR << "Building index for " << file.id_ << " failed: " << status.ToString();
+//        }
+//
+//        if (shutting_down_.load(std::memory_order_acquire)) {
+//            ENGINE_LOG_DEBUG << "Server will shutdown, skip build index action";
+//            break;
+//        }
+//    }
+
     ENGINE_LOG_TRACE << "Background build index thread exit";
 }
 

cpp/src/db/engine/ExecutionEngineImpl.h

@@ -59,7 +59,7 @@ class ExecutionEngineImpl : public ExecutionEngine {
     CopyToGpu(uint64_t device_id) override;
 
     Status
-    CopyToIndexFileToGpu(uint64_t device_id) override; 
+    CopyToIndexFileToGpu(uint64_t device_id) override;
 
     Status
     CopyToCpu() override;

cpp/src/scheduler/job/BuildIndexJob.cpp

@@ -15,16 +15,14 @@
 // specific language governing permissions and limitations
 // under the License.
 
-#include "BuildIndexJob.h"
 #include "utils/Log.h"
-
+#include "BuildIndexJob.h"
 
 namespace milvus {
 namespace scheduler {
 
 BuildIndexJob::BuildIndexJob(JobId id, engine::meta::MetaPtr meta_ptr, engine::DBOptions options)
     : Job(id, JobType::BUILD), meta_ptr_(std::move(meta_ptr)), options_(std::move(options)) {
-
 }
 
 bool
@@ -54,6 +52,5 @@ BuildIndexJob::BuildIndexDone(size_t to_index_id) {
     SERVER_LOG_DEBUG << "BuildIndexJob " << id() << " finish index file: " << to_index_id;
 }
 
-
-}
-}
\ No newline at end of file
+}  // namespace scheduler
+}  // namespace milvus

cpp/src/scheduler/job/BuildIndexJob.h

@@ -87,5 +87,5 @@ class BuildIndexJob : public Job {
 
 using BuildIndexJobPtr = std::shared_ptr<BuildIndexJob>;
 
-}
-}
+}  // namespace scheduler
+}  // namespace milvus

cpp/src/scheduler/task/BuildIndexTask.cpp

@@ -26,6 +26,7 @@
 #include <thread>
 #include <utility>
 
+
 namespace milvus {
 namespace scheduler {
 
@@ -61,7 +62,7 @@ XBuildIndexTask::Load(milvus::scheduler::LoadType type, uint8_t device_id) {
                 error_msg = "Wrong load type";
                 stat = Status(SERVER_UNEXPECTED_ERROR, error_msg);
             }
-        } catch (std::exception& ex) {
+        } catch (std::exception &ex) {
             // typical error: out of disk space or permition denied
             error_msg = "Failed to load to_index file: " + std::string(ex.what());
             stat = Status(SERVER_UNEXPECTED_ERROR, error_msg);
@@ -69,7 +70,7 @@ XBuildIndexTask::Load(milvus::scheduler::LoadType type, uint8_t device_id) {
 
         if (!stat.ok()) {
             Status s;
-            if(stat.ToString().find("out of memory") != std::string::npos) {
+            if (stat.ToString().find("out of memory") != std::string::npos) {
                 error_msg = "out of memory: " + type_str;
                 s = Status(SERVER_UNEXPECTED_ERROR, error_msg);
             } else {
@@ -108,7 +109,7 @@ XBuildIndexTask::Execute() {
     if (auto job = job_.lock()) {
         auto build_index_job = std::static_pointer_cast<scheduler::BuildIndexJob>(job);
         std::string location = file_->location_;
-        EngineType engine_type = (EngineType)file_->engine_type_;
+        EngineType engine_type = (EngineType) file_->engine_type_;
         std::shared_ptr<engine::ExecutionEngine> index;
 
         // step 2: create table file
@@ -116,7 +117,7 @@ XBuildIndexTask::Execute() {
         table_file.table_id_ = file_->table_id_;
         table_file.date_ = file_->date_;
         table_file.file_type_ =
-            engine::meta::TableFileSchema::NEW_INDEX;  // for multi-db-path, distribute index file averagely to each path
+            engine::meta::TableFileSchema::NEW_INDEX;
 
         engine::meta::MetaPtr meta_ptr = build_index_job->meta();
         Status status = build_index_job->meta()->CreateTableFile(table_file);
@@ -204,7 +205,8 @@ XBuildIndexTask::Execute() {
 
             table_file.file_type_ = engine::meta::TableFileSchema::TO_DELETE;
             status = meta_ptr->UpdateTableFile(table_file);
-            ENGINE_LOG_DEBUG << "Failed to up  date file to index, mark file: " << table_file.file_id_ << " to to_delete";
+            ENGINE_LOG_DEBUG << "Failed to up  date file to index, mark file: " << table_file.file_id_
+                             << " to to_delete";
         }
 
         build_index_job->BuildIndexDone(to_index_id_);

cpp/src/scheduler/task/BuildIndexTask.h

@@ -41,7 +41,6 @@ class XBuildIndexTask : public Task {
     size_t to_index_id_ = 0;
     int to_index_type_ = 0;
     ExecutionEnginePtr to_index_engine_ = nullptr;
-
 };
 
 }  // namespace scheduler

cpp/unittest/metrics/utils.cpp

@@ -66,6 +66,7 @@ ms::engine::DBOptions MetricTest::GetOptions() {
 }
 
 void MetricTest::SetUp() {
+    boost::filesystem::remove_all("/tmp/milvus_test");
     InitLog();
     auto options = GetOptions();
     db_ = ms::engine::DBFactory::Build(options);