KeDeng has posted comments on this change. ( http://gerrit.cloudera.org:8080/20178 )
Change subject: KUDU-3326 disable compaction for soft-deleted table ...................................................................... Patch Set 7: (2 comments) Thanks for your reviews. http://gerrit.cloudera.org:8080/#/c/20178/5/src/kudu/master/catalog_manager.h File src/kudu/master/catalog_manager.h: http://gerrit.cloudera.org:8080/#/c/20178/5/src/kudu/master/catalog_manager.h@713 PS5, Line 713: // The RPC context is provided for logging/tracing purposes, > How about use an enum instead of boolean to clarify the meaning? Done http://gerrit.cloudera.org:8080/#/c/20178/5/src/kudu/master/catalog_manager.cc File src/kudu/master/catalog_manager.cc: http://gerrit.cloudera.org:8080/#/c/20178/5/src/kudu/master/catalog_manager.cc@6039 PS5, Line 6039: and remove > The comments need to be updated. Done -- To view, visit http://gerrit.cloudera.org:8080/20178 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: kudu Gerrit-Branch: master Gerrit-MessageType: comment Gerrit-Change-Id: I060810051613a19e6f5c6506effda2d698528839 Gerrit-Change-Number: 20178 Gerrit-PatchSet: 7 Gerrit-Owner: KeDeng <[email protected]> Gerrit-Reviewer: KeDeng <[email protected]> Gerrit-Reviewer: Kudu Jenkins (120) Gerrit-Reviewer: Yingchun Lai <[email protected]> Gerrit-Comment-Date: Mon, 21 Aug 2023 07:29:09 +0000 Gerrit-HasComments: Yes
