Adar Dembo has posted comments on this change. ( http://gerrit.cloudera.org:8080/12907 )
Change subject: remove schema copies from RowBlock and RowBuilder ...................................................................... Patch Set 5: (1 comment) http://gerrit.cloudera.org:8080/#/c/12907/5/src/kudu/common/rowblock.h File src/kudu/common/rowblock.h: http://gerrit.cloudera.org:8080/#/c/12907/5/src/kudu/common/rowblock.h@195 PS5, Line 195: RowBlock(const Schema* schema, > mind adding a comment just to indicate the lifetime requirements of 'schema Done -- To view, visit http://gerrit.cloudera.org:8080/12907 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: kudu Gerrit-Branch: master Gerrit-MessageType: comment Gerrit-Change-Id: Ie4d60640b2bac4f2b3077ddd696966e1f6658740 Gerrit-Change-Number: 12907 Gerrit-PatchSet: 5 Gerrit-Owner: Adar Dembo <[email protected]> Gerrit-Reviewer: Adar Dembo <[email protected]> Gerrit-Reviewer: Mike Percy <[email protected]> Gerrit-Reviewer: Tidy Bot (241) Gerrit-Reviewer: Todd Lipcon <[email protected]> Gerrit-Comment-Date: Tue, 09 Apr 2019 20:45:03 +0000 Gerrit-HasComments: Yes
