From 945d006d8da6286c6974e56f6e91eabb7fa67a1f Mon Sep 17 00:00:00 2001
From: Yudong Cai <yudong.cai@zilliz.com>
Date: Wed, 13 Dec 2023 10:45:24 +0800
Subject: [PATCH] Remove useless prometheus metrics

Signed-off-by: Yudong Cai <yudong.cai@zilliz.com>
---
 include/knowhere/prometheus_client.h | 7 +------
 src/common/index.cc                  | 5 -----
 src/common/prometheus_client.cc      | 7 +------
 3 files changed, 2 insertions(+), 17 deletions(-)

diff --git a/include/knowhere/prometheus_client.h b/include/knowhere/prometheus_client.h
index 8a437d56a..dcfa309b2 100644
--- a/include/knowhere/prometheus_client.h
+++ b/include/knowhere/prometheus_client.h
@@ -74,13 +74,8 @@ extern const std::unique_ptr<PrometheusClient> prometheusClient;
 #define DECLARE_PROMETHEUS_COUNTER(name_counter) extern prometheus::Counter& name_counter;
 #define DECLARE_PROMETHEUS_HISTOGRAM(name_histogram) extern prometheus::Histogram& name_histogram;
 
-DECLARE_PROMETHEUS_COUNTER(knowhere_build_count);
-DECLARE_PROMETHEUS_COUNTER(knowhere_search_count);
-DECLARE_PROMETHEUS_COUNTER(knowhere_ann_iterator_count);
-DECLARE_PROMETHEUS_COUNTER(knowhere_range_search_count);
 DECLARE_PROMETHEUS_HISTOGRAM(knowhere_build_latency);
-DECLARE_PROMETHEUS_HISTOGRAM(knowhere_search_topk);
 DECLARE_PROMETHEUS_HISTOGRAM(knowhere_search_latency);
-DECLARE_PROMETHEUS_HISTOGRAM(knowhere_ann_iterator_init_latency);
 DECLARE_PROMETHEUS_HISTOGRAM(knowhere_range_search_latency);
+DECLARE_PROMETHEUS_HISTOGRAM(knowhere_ann_iterator_init_latency);
 }  // namespace knowhere
diff --git a/src/common/index.cc b/src/common/index.cc
index 2c7de7e8a..c43279064 100644
--- a/src/common/index.cc
+++ b/src/common/index.cc
@@ -44,7 +44,6 @@ Index<T>::Build(const DataSet& dataset, const Json& json) {
     auto span = rc.ElapseFromBegin("done");
     span *= 0.000001;  // convert to s
     knowhere_build_latency.Observe(span);
-    knowhere_build_count.Increment();
 #else
     auto res = this->node->Build(dataset, *cfg);
 #endif
@@ -83,8 +82,6 @@ Index<T>::Search(const DataSet& dataset, const Json& json, const BitsetView& bit
     auto span = rc.ElapseFromBegin("done");
     span *= 0.001;  // convert to ms
     knowhere_search_latency.Observe(span);
-    knowhere_search_count.Increment();
-    knowhere_search_topk.Observe(cfg->k.value());
 #else
     auto res = this->node->Search(dataset, *cfg, bitset);
 #endif
@@ -108,7 +105,6 @@ Index<T>::AnnIterator(const DataSet& dataset, const Json& json, const BitsetView
     auto span = rc.ElapseFromBegin("done");
     span *= 0.001;  // convert to ms
     knowhere_search_latency.Observe(span);
-    knowhere_ann_iterator_count.Increment();
 #else
     auto res = this->node->AnnIterator(dataset, *cfg, bitset);
 #endif
@@ -131,7 +127,6 @@ Index<T>::RangeSearch(const DataSet& dataset, const Json& json, const BitsetView
     auto span = rc.ElapseFromBegin("done");
     span *= 0.001;  // convert to ms
     knowhere_range_search_latency.Observe(span);
-    knowhere_range_search_count.Increment();
 #else
     auto res = this->node->RangeSearch(dataset, *cfg, bitset);
 #endif
diff --git a/src/common/prometheus_client.cc b/src/common/prometheus_client.cc
index 83b6a21cc..0475eae82 100644
--- a/src/common/prometheus_client.cc
+++ b/src/common/prometheus_client.cc
@@ -25,14 +25,9 @@ const std::unique_ptr<PrometheusClient> prometheusClient = std::make_unique<Prom
  *   An error has occurred while serving metrics:
  *   text format parsing error in line 50: expected float as value, got "=\"0.9\"}"
  ******************************************************************************/
-DEFINE_PROMETHEUS_COUNTER(knowhere_build_count, "knowhere index build count")
-DEFINE_PROMETHEUS_COUNTER(knowhere_search_count, "knowhere search count")
-DEFINE_PROMETHEUS_COUNTER(knowhere_ann_iterator_count, "knowhere ann iterator count")
-DEFINE_PROMETHEUS_COUNTER(knowhere_range_search_count, "knowhere range search count")
 DEFINE_PROMETHEUS_HISTOGRAM(knowhere_build_latency, "index build latency in knowhere (s)")
-DEFINE_PROMETHEUS_HISTOGRAM(knowhere_search_topk, "knowhere search topk")
 DEFINE_PROMETHEUS_HISTOGRAM(knowhere_search_latency, "search latency in knowhere (ms)")
-DEFINE_PROMETHEUS_HISTOGRAM(knowhere_ann_iterator_init_latency, "ann iterator init latency in knowhere (ms)")
 DEFINE_PROMETHEUS_HISTOGRAM(knowhere_range_search_latency, "range search latency in knowhere (ms)")
+DEFINE_PROMETHEUS_HISTOGRAM(knowhere_ann_iterator_init_latency, "ann iterator init latency in knowhere (ms)")
 
 }  // namespace knowhere