Andrew Wong has posted comments on this change. ( http://gerrit.cloudera.org:8080/15802 )
Change subject: KUDU-2844 (3/3): avoid copying plain/dict strings to RowBlock Arena ...................................................................... Patch Set 3: Code-Review+1 (3 comments) http://gerrit.cloudera.org:8080/#/c/15802/3//COMMIT_MSG Commit Message: http://gerrit.cloudera.org:8080/#/c/15802/3//COMMIT_MSG@42 PS3, Line 42: I waited until the data had been fully flushed from MRS before running Any chance compactions could have occurred and interfered with the block layout? http://gerrit.cloudera.org:8080/#/c/15802/3/src/kudu/common/rowblock_memory.h File src/kudu/common/rowblock_memory.h: http://gerrit.cloudera.org:8080/#/c/15802/3/src/kudu/common/rowblock_memory.h@46 PS3, Line 46: template<class T> : void RetainReference(T* item) { We're both storing and passing RowBlockRefCounted*s, but the callsites might refer to derived refcounters, so we're templating here so we don't have to cast, right? http://gerrit.cloudera.org:8080/#/c/15802/3/src/kudu/integration-tests/full_stack-insert-scan-test.cc File src/kudu/integration-tests/full_stack-insert-scan-test.cc: http://gerrit.cloudera.org:8080/#/c/15802/3/src/kudu/integration-tests/full_stack-insert-scan-test.cc@204 PS3, Line 204: // Enable fault tolerance. nit: would be nice to mention what's so special about fault tolerant scans. What comes to mind w.r.t this patch is that fault tolerance means we're doing an ordered scan, which exercises different iterator codepaths. Is that the intent, or is there something else special about ft scans? -- To view, visit http://gerrit.cloudera.org:8080/15802 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: kudu Gerrit-Branch: master Gerrit-MessageType: comment Gerrit-Change-Id: I93fa1f9fd401814a42dc5a1f3fd2ffb1286ac441 Gerrit-Change-Number: 15802 Gerrit-PatchSet: 3 Gerrit-Owner: Todd Lipcon <[email protected]> Gerrit-Reviewer: Andrew Wong <[email protected]> Gerrit-Reviewer: Andrew Wong <[email protected]> Gerrit-Reviewer: Kudu Jenkins (120) Gerrit-Reviewer: Tidy Bot (241) Gerrit-Reviewer: Todd Lipcon <[email protected]> Gerrit-Reviewer: Volodymyr Verovkin <[email protected]> Gerrit-Comment-Date: Fri, 01 May 2020 05:55:23 +0000 Gerrit-HasComments: Yes
