Adar Dembo has posted comments on this change. Change subject: rowset_metadata: use flat_map for column->block mapping ......................................................................
Patch Set 1: (1 comment) http://gerrit.cloudera.org:8080/#/c/6470/1/src/kudu/tablet/rowset_metadata.h File src/kudu/tablet/rowset_metadata.h: Line 68: typedef boost::container::flat_map<ColumnId, BlockId> ColumnIdToBlockIdMap; Nit: add a comment so no one accidentally replaces this with std::map. -- To view, visit http://gerrit.cloudera.org:8080/6470 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-MessageType: comment Gerrit-Change-Id: I1b7eaac3481672c69709009b6eacc2008027d4cd Gerrit-PatchSet: 1 Gerrit-Project: kudu Gerrit-Branch: master Gerrit-Owner: Todd Lipcon <[email protected]> Gerrit-Reviewer: Adar Dembo <[email protected]> Gerrit-Reviewer: Kudu Jenkins Gerrit-HasComments: Yes
