Yingchun Lai has posted comments on this change. ( http://gerrit.cloudera.org:8080/19670 )
Change subject: [GC] gc ancient, fully deleted rowsets without live row count ...................................................................... Patch Set 7: Code-Review+1 (6 comments) http://gerrit.cloudera.org:8080/#/c/19670/5/src/kudu/tablet/diskrowset-test.cc File src/kudu/tablet/diskrowset-test.cc: http://gerrit.cloudera.org:8080/#/c/19670/5/src/kudu/tablet/diskrowset-test.cc@771 PS5, Line 771: TestGCDRSWithoutLiveRowCount) { > Done Done http://gerrit.cloudera.org:8080/#/c/19670/5/src/kudu/tablet/diskrowset-test.cc@773 PS5, Line 773: WriteTestRowSet(); > Well. I think there may be a misunderstanding here. I see, thanks! http://gerrit.cloudera.org:8080/#/c/19670/5/src/kudu/tablet/diskrowset-test.cc@780 PS5, Line 780: CHECK_OK(rs->CountLiveRows(&rs_live_rows)); > Done Done http://gerrit.cloudera.org:8080/#/c/19670/5/src/kudu/tablet/diskrowset-test.cc@790 PS5, Line 790: // The live rows count is 0 if the DRS has been fully deleted without DMS exist. > Done Done http://gerrit.cloudera.org:8080/#/c/19670/5/src/kudu/tablet/diskrowset-test.cc@801 PS5, Line 801: WriteTestRowSet(); > Considering that the data in DMS may not be recorded, I added 'ASSERT'_ GE Done http://gerrit.cloudera.org:8080/#/c/19670/5/src/kudu/tablet/diskrowset-test.cc@802 PS5, Line 802: kRowSet> rs; > Removed. Done -- To view, visit http://gerrit.cloudera.org:8080/19670 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: kudu Gerrit-Branch: master Gerrit-MessageType: comment Gerrit-Change-Id: Iacdff107b8b07cbd56f47f296a93f4bcfbf56b41 Gerrit-Change-Number: 19670 Gerrit-PatchSet: 7 Gerrit-Owner: KeDeng <[email protected]> Gerrit-Reviewer: KeDeng <[email protected]> Gerrit-Reviewer: Kudu Jenkins (120) Gerrit-Reviewer: Tidy Bot (241) Gerrit-Reviewer: Yingchun Lai <[email protected]> Gerrit-Comment-Date: Tue, 04 Apr 2023 15:46:11 +0000 Gerrit-HasComments: Yes
