[opt](segment) Address review comments

- Add rowset_id to LazyInitSegmentIterator skip log for better debuggability
- Remove unused #include "common/config.h" from lazy_init_segment_iterator.h
- Use specific DebugPoints::remove() instead of clear() in test TearDown
- Assert json2pb::JsonToProtoMessage return value in test

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>

Author: dataroaring

be/src/storage/segment/lazy_init_segment_iterator.cpp

@@ -43,7 +43,8 @@ Status LazyInitSegmentIterator::init(const StorageReadOptions& opts) {
         auto st = SegmentLoader::instance()->load_segment(
                 _rowset, _segment_id, &segment_cache_handle, _should_use_cache, false, opts.stats);
         if (st.is<ErrorCode::NOT_FOUND>() && config::ignore_not_found_segment) {
-            LOG(WARNING) << "segment not found, skip it. seg_id=" << _segment_id;
+            LOG(WARNING) << "segment not found, skip it. rowset_id=" << _rowset->rowset_id()
+                         << ", seg_id=" << _segment_id;
             // _inner_iterator remains nullptr, next_batch() will return EOF
             return Status::OK();
         }

be/src/storage/segment/lazy_init_segment_iterator.h

@@ -17,7 +17,6 @@
 
 #pragma once
 
-#include "common/config.h"
 #include "core/block/block.h"
 #include "storage/segment/common.h"
 #include "storage/segment/segment.h"

be/test/storage/segment/ignore_not_found_segment_test.cpp

@@ -44,7 +44,7 @@ class IgnoreNotFoundSegmentTest : public testing::Test {
     }
 
     void TearDown() override {
-        DebugPoints::instance()->clear();
+        DebugPoints::instance()->remove("BetaRowset::load_segment.return_not_found");
         config::ignore_not_found_segment = _saved_ignore;
         config::enable_debug_points = _saved_debug_points;
 
@@ -76,7 +76,7 @@ class IgnoreNotFoundSegmentTest : public testing::Test {
             "empty": false
         })";
         RowsetMetaPB pb;
-        json2pb::JsonToProtoMessage(json, &pb);
+        EXPECT_TRUE(json2pb::JsonToProtoMessage(json, &pb));
         pb.set_start_version(0);
         pb.set_end_version(1);
         pb.set_num_segments(num_segments);