This is an automated email from the ASF dual-hosted git repository.
gavinchou pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/doris.git
The following commit(s) were added to refs/heads/master by this push:
new c6a39069c68 [fix](cloud) get_tablet_merged_stats should returns the
tablet idx (#55589)
c6a39069c68 is described below
commit c6a39069c68731e053aafe87161bdceb4f5fff61
Author: walter <[email protected]>
AuthorDate: Tue Sep 2 21:29:13 2025 +0800
[fix](cloud) get_tablet_merged_stats should returns the tablet idx (#55589)
---
cloud/src/meta-store/meta_reader.cpp | 5 +++++
cloud/test/meta_service_versioned_read_test.cpp | 7 +++++++
2 files changed, 12 insertions(+)
diff --git a/cloud/src/meta-store/meta_reader.cpp
b/cloud/src/meta-store/meta_reader.cpp
index b58b0639b4c..90ee3370b46 100644
--- a/cloud/src/meta-store/meta_reader.cpp
+++ b/cloud/src/meta-store/meta_reader.cpp
@@ -354,6 +354,11 @@ TxnErrorCode
MetaReader::get_tablet_merged_stats(Transaction* txn, int64_t table
tablet_stats->set_data_size(load_stats.data_size() +
compact_stats.data_size());
tablet_stats->set_index_size(load_stats.index_size() +
compact_stats.index_size());
tablet_stats->set_segment_size(load_stats.segment_size() +
compact_stats.segment_size());
+ if (load_stats.has_idx()) {
+ tablet_stats->mutable_idx()->CopyFrom(load_stats.idx());
+ } else if (compact_stats.has_idx()) {
+ tablet_stats->mutable_idx()->CopyFrom(compact_stats.idx());
+ }
}
Versionstamp read_version = std::min(load_version, compact_version);
if (versionstamp) {
diff --git a/cloud/test/meta_service_versioned_read_test.cpp
b/cloud/test/meta_service_versioned_read_test.cpp
index c770bdc2b95..4af4b441342 100644
--- a/cloud/test/meta_service_versioned_read_test.cpp
+++ b/cloud/test/meta_service_versioned_read_test.cpp
@@ -768,6 +768,13 @@ TEST(MetaServiceVersionedReadTest, GetTabletStats) {
EXPECT_EQ(res.tablet_stats(0).num_segments(), 0);
EXPECT_EQ(res.tablet_stats(0).index_size(), 0);
EXPECT_EQ(res.tablet_stats(0).segment_size(), 0);
+
+ // The tablet idx should be set.
+ auto tablet_stats = res.tablet_stats(0);
+ EXPECT_EQ(tablet_stats.idx().tablet_id(), tablet_id);
+ EXPECT_EQ(tablet_stats.idx().index_id(), index_id);
+ EXPECT_EQ(tablet_stats.idx().partition_id(), partition_id);
+ EXPECT_EQ(tablet_stats.idx().table_id(), table_id);
}
{
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
