Merge branch 'branch-0.4.0' into 'branch-0.4.0'

fix typo

See merge request megasearch/milvus!462

Former-commit-id: 245bded1d2d1eec1f051d91a6f74f72bbbcff693

cpp/build.sh

@@ -88,7 +88,7 @@ pushd `pwd`/src/core
 popd
 
 cd cmake_build
-git
+
 CUDA_COMPILER=/usr/local/cuda/bin/nvcc
 
 if [[ ${MAKE_CLEAN} == "ON" ]]; then

cpp/src/metrics/Metrics.h

@@ -37,21 +37,22 @@ public:
     }
 
     ~CollectInsertMetrics() {
-        auto end_time = METRICS_NOW_TIME;
-        auto total_time = METRICS_MICROSECONDS(start_time_, end_time);
-        double avg_time = total_time / n_;
-        for (int i = 0; i < n_; ++i) {
-            Metrics::GetInstance().AddVectorsDurationHistogramOberve(avg_time);
-        }
+        if(n_ > 0) {
+            auto end_time = METRICS_NOW_TIME;
+            auto total_time = METRICS_MICROSECONDS(start_time_, end_time);
+            double avg_time = total_time / n_;
+            for (int i = 0; i < n_; ++i) {
+                Metrics::GetInstance().AddVectorsDurationHistogramOberve(avg_time);
+            }
 
-        //    server::Metrics::GetInstance().add_vector_duration_seconds_quantiles().Observe((average_time));
-        if (status_.ok()) {
-            server::Metrics::GetInstance().AddVectorsSuccessTotalIncrement(n_);
-            server::Metrics::GetInstance().AddVectorsSuccessGaugeSet(n_);
-        }
-        else {
-            server::Metrics::GetInstance().AddVectorsFailTotalIncrement(n_);
-            server::Metrics::GetInstance().AddVectorsFailGaugeSet(n_);
+            //    server::Metrics::GetInstance().add_vector_duration_seconds_quantiles().Observe((average_time));
+            if (status_.ok()) {
+                server::Metrics::GetInstance().AddVectorsSuccessTotalIncrement(n_);
+                server::Metrics::GetInstance().AddVectorsSuccessGaugeSet(n_);
+            } else {
+                server::Metrics::GetInstance().AddVectorsFailTotalIncrement(n_);
+                server::Metrics::GetInstance().AddVectorsFailGaugeSet(n_);
+            }
         }
     }
 
@@ -69,14 +70,16 @@ public:
     }
 
     ~CollectQueryMetrics() {
-        auto end_time = METRICS_NOW_TIME;
-        auto total_time = METRICS_MICROSECONDS(start_time_, end_time);
-        for (int i = 0; i < nq_; ++i) {
-            server::Metrics::GetInstance().QueryResponseSummaryObserve(total_time);
+        if(nq_ > 0) {
+            auto end_time = METRICS_NOW_TIME;
+            auto total_time = METRICS_MICROSECONDS(start_time_, end_time);
+            for (int i = 0; i < nq_; ++i) {
+                server::Metrics::GetInstance().QueryResponseSummaryObserve(total_time);
+            }
+            auto average_time = total_time / nq_;
+            server::Metrics::GetInstance().QueryVectorResponseSummaryObserve(average_time, nq_);
+            server::Metrics::GetInstance().QueryVectorResponsePerSecondGaugeSet(double(nq_) / total_time);
         }
-        auto average_time = total_time / nq_;
-        server::Metrics::GetInstance().QueryVectorResponseSummaryObserve(average_time, nq_);
-        server::Metrics::GetInstance().QueryVectorResponsePerSecondGaugeSet(double (nq_) / total_time);
     }
 
 private: