github-actions[bot] commented on code in PR #15030: URL: https://github.com/apache/doris/pull/15030#discussion_r1046688705
########## be/test/olap/tablet_test.cpp: ########## @@ -252,6 +272,41 @@ TEST_F(TestTablet, delete_expired_stale_rowset) { _tablet.reset(); } +TEST_F(TestTablet, pad_rowset) { + std::vector<RowsetMetaSharedPtr> rs_metas; + auto ptr1 = std::make_shared<RowsetMeta>(); + init_rs_meta(ptr1, 1, 2); + rs_metas.push_back(ptr1); + RowsetSharedPtr rowset1 = make_shared<BetaRowset>(nullptr, "", ptr1); + + auto ptr2 = std::make_shared<RowsetMeta>(); + init_rs_meta(ptr2, 3, 4); + rs_metas.push_back(ptr2); + RowsetSharedPtr rowset2 = make_shared<BetaRowset>(nullptr, "", ptr2); + + auto ptr3 = std::make_shared<RowsetMeta>(); + init_rs_meta(ptr3, 6, 7); + rs_metas.push_back(ptr3); + RowsetSharedPtr rowset3 = make_shared<BetaRowset>(nullptr, "", ptr3); + + for (auto& rowset : rs_metas) { + _tablet_meta->add_rs_meta(rowset); + } + + _data_dir->init(); + TabletSharedPtr _tablet(new Tablet(_tablet_meta, _data_dir.get())); + _tablet->init(); + + Version version(5, 5); + std::vector<RowsetReaderSharedPtr> readers; + ASSERT_FALSE(_tablet->capture_rs_readers(version, &readers).ok()); + readers.clear(); + + PadRowsetAction action; + action._pad_rowset(_tablet, version); Review Comment: warning: '_pad_rowset' is a private member of 'doris::PadRowsetAction' [clang-diagnostic-error] ```cpp action._pad_rowset(_tablet, version); ^ ``` **be/src/http/action/pad_rowset_action.h:40:** declared private here ```cpp Status _pad_rowset(TabletSharedPtr tablet, const Version& version); ^ ``` -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: commits-unsubscr...@doris.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org --------------------------------------------------------------------- To unsubscribe, e-mail: commits-unsubscr...@doris.apache.org For additional commands, e-mail: commits-h...@doris.apache.org