Alexey Serbin has submitted this change and it was merged. ( http://gerrit.cloudera.org:8080/19333 )
Change subject: [common] small optimisation on PartitionContainsRow ...................................................................... [common] small optimisation on PartitionContainsRow This patch updates PartitionSchema::PartitionContainsRow() to avoid calling EncodeColumns() twice for the same data. That's to improve performance of Tablet::CheckRowInTablet() which is in a hot path when inserting data into a tablet. Change-Id: Iaf1d8a9ea533a859d218292a6a0e2a0818781111 Reviewed-on: http://gerrit.cloudera.org:8080/19333 Tested-by: Alexey Serbin <[email protected]> Reviewed-by: Yingchun Lai <[email protected]> --- M src/kudu/common/partition.cc M src/kudu/common/partition.h 2 files changed, 16 insertions(+), 8 deletions(-) Approvals: Alexey Serbin: Verified Yingchun Lai: Looks good to me, approved -- To view, visit http://gerrit.cloudera.org:8080/19333 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: kudu Gerrit-Branch: master Gerrit-MessageType: merged Gerrit-Change-Id: Iaf1d8a9ea533a859d218292a6a0e2a0818781111 Gerrit-Change-Number: 19333 Gerrit-PatchSet: 2 Gerrit-Owner: Alexey Serbin <[email protected]> Gerrit-Reviewer: Alexey Serbin <[email protected]> Gerrit-Reviewer: Attila Bukor <[email protected]> Gerrit-Reviewer: Kudu Jenkins (120) Gerrit-Reviewer: Yingchun Lai <[email protected]>
