Yingchun Lai has posted comments on this change. ( http://gerrit.cloudera.org:8080/17871 )
Change subject: KUDU-3318 [LBM] Runtime compact log container metadata ...................................................................... Patch Set 6: (4 comments) http://gerrit.cloudera.org:8080/#/c/17871/3/src/kudu/fs/log_block_manager-test.cc File src/kudu/fs/log_block_manager-test.cc: http://gerrit.cloudera.org:8080/#/c/17871/3/src/kudu/fs/log_block_manager-test.cc@1253 PS3, Line 1253: 32 * 102 > nit: prefer to be more explicit with these values (e.g. 32 * 1024), optimiz Done http://gerrit.cloudera.org:8080/#/c/17871/3/src/kudu/fs/log_block_manager-test.cc@1254 PS3, Line 1254: NO_FATALS(mt_create_and_delete > nit: wrap in NO_FATALS() so the error messages point to this line number if Done http://gerrit.cloudera.org:8080/#/c/17871/5/src/kudu/fs/log_block_manager.cc File src/kudu/fs/log_block_manager.cc: http://gerrit.cloudera.org:8080/#/c/17871/5/src/kudu/fs/log_block_manager.cc@864 PS5, Line 864: std::move(live_blo > We should std::move() this to avoid a copy. Done http://gerrit.cloudera.org:8080/#/c/17871/5/src/kudu/fs/log_block_manager.cc@2812 PS5, Line 2812: vector<BlockRecordPB> records = LogBlockContainer::SortRecords(std::move(live_block_records)); > We should std::move() this to avoid a copy. Done -- To view, visit http://gerrit.cloudera.org:8080/17871 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: kudu Gerrit-Branch: master Gerrit-MessageType: comment Gerrit-Change-Id: I72bdf73fa73403918c01dfa9d45530fad47ecaf6 Gerrit-Change-Number: 17871 Gerrit-PatchSet: 6 Gerrit-Owner: Yingchun Lai <[email protected]> Gerrit-Reviewer: Andrew Wong <[email protected]> Gerrit-Reviewer: Kudu Jenkins (120) Gerrit-Reviewer: Tidy Bot (241) Gerrit-Reviewer: Yingchun Lai <[email protected]> Gerrit-Comment-Date: Tue, 12 Oct 2021 02:49:38 +0000 Gerrit-HasComments: Yes
