[doris] branch branch-2.0 updated: [Enhancement](inverted index) remove time-consuming log prints in bkd reader (#23381) (#23412)

[kxiao]

This is an automated email from the ASF dual-hosted git repository.

kxiao pushed a commit to branch branch-2.0
in repository https://gitbox.apache.org/repos/asf/doris.git


The following commit(s) were added to refs/heads/branch-2.0 by this push:
     new 80572edb8a [Enhancement](inverted index) remove time-consuming log 
prints in bkd reader (#23381) (#23412)
80572edb8a is described below

commit 80572edb8ab5a7fe319850fe2a9552dd93caf2fd
Author: airborne12 <airborn...@gmail.com>
AuthorDate: Thu Aug 24 12:31:52 2023 +0800

    [Enhancement](inverted index) remove time-consuming log prints in bkd 
reader (#23381) (#23412)
---
 be/src/olap/rowset/segment_v2/inverted_index_reader.cpp | 11 +++--------
 1 file changed, 3 insertions(+), 8 deletions(-)

diff --git a/be/src/olap/rowset/segment_v2/inverted_index_reader.cpp 
b/be/src/olap/rowset/segment_v2/inverted_index_reader.cpp
index 22f5abd0c1..99e0fe1f1f 100644
--- a/be/src/olap/rowset/segment_v2/inverted_index_reader.cpp
+++ b/be/src/olap/rowset/segment_v2/inverted_index_reader.cpp
@@ -613,7 +613,6 @@ Status BkdIndexReader::bkd_query(OlapReaderStatistics* 
stats, const std::string&
 Status BkdIndexReader::try_query(OlapReaderStatistics* stats, const 
std::string& column_name,
                                  const void* query_value, 
InvertedIndexQueryType query_type,
                                  uint32_t* count) {
-    uint64_t start = UnixMillis();
     auto visitor = std::make_unique<InvertedIndexVisitor>(nullptr, query_type, 
true);
     std::shared_ptr<lucene::util::bkd::bkd_reader> r;
     try {
@@ -632,8 +631,7 @@ Status BkdIndexReader::try_query(OlapReaderStatistics* 
stats, const std::string&
                 "BKD Query CLuceneError Occurred, error msg: {}", e.what());
     }
 
-    LOG(INFO) << "BKD index try search time taken: " << UnixMillis() - start 
<< "ms "
-              << " column: " << column_name << " result: " << *count;
+    VLOG_DEBUG << "BKD index try search column: " << column_name << " result: 
" << *count;
     return Status::OK();
 }
 
@@ -663,11 +661,9 @@ Status BkdIndexReader::query(OlapReaderStatistics* stats, 
const std::string& col
     //     stats->inverted_index_query_cache_miss++;
     // }
 
-    uint64_t start = UnixMillis();
     auto visitor = std::make_unique<InvertedIndexVisitor>(bit_map, query_type);
     std::shared_ptr<lucene::util::bkd::bkd_reader> r;
     try {
-        SCOPED_RAW_TIMER(&stats->inverted_index_searcher_search_timer);
         auto st = bkd_query(stats, column_name, query_value, query_type, r, 
visitor.get());
         if (!st.ok()) {
             if (st.code() == ErrorCode::END_OF_FILE) {
@@ -687,9 +683,8 @@ Status BkdIndexReader::query(OlapReaderStatistics* stats, 
const std::string& col
     // term_match_bitmap->runOptimize();
     // cache->insert(cache_key, term_match_bitmap, &cache_handle);
 
-    LOG(INFO) << "BKD index search time taken: " << UnixMillis() - start << 
"ms "
-              << " column: " << column_name << " result: " << 
bit_map->cardinality()
-              << " reader stats: " << r->stats.to_string();
+    VLOG_DEBUG << "BKD index search column: " << column_name
+               << " result: " << bit_map->cardinality();
     return Status::OK();
 }
 


---------------------------------------------------------------------
To unsubscribe, e-mail: commits-unsubscr...@doris.apache.org
For additional commands, e-mail: commits-h...@doris.apache.org