You are viewing a plain text version of this content. The canonical link for it is here.
Posted to reviews@kudu.apache.org by "Todd Lipcon (Code Review)" <ge...@cloudera.org> on 2017/03/28 23:44:35 UTC

[kudu-CR] rowset tree: add bulk queries

Hello Kudu Jenkins,

I'd like you to reexamine a change.  Please visit

    http://gerrit.cloudera.org:8080/6482

to look at the new patch set (#4).

Change subject: rowset_tree: add bulk queries
......................................................................

rowset_tree: add bulk queries

This extends the bulk query API from IntervalTree up into RowSet.

TestRowSetTree.TestPerformance shows a 6-7x improvement for bulk query vs
one-by-one:

I0328 15:36:15.330320 30901 rowset_tree-test.cc:126] Time spent Querying rowset 1000000 times: real 0.344s	user 0.344s	sys 0.000s
I0328 15:36:15.701807 30901 rowset_tree-test.cc:142] Time spent Querying rowset 1000000 times in batch: real 0.045s	user 0.044s	sys 0.000s

More importantly, this will allow for other higher-level optimizations.

Change-Id: I6ab24681dfbb3b1e6f08d52eb0647a5f3ca6851f
---
M src/kudu/tablet/rowset_tree-test.cc
M src/kudu/tablet/rowset_tree.cc
M src/kudu/tablet/rowset_tree.h
3 files changed, 85 insertions(+), 5 deletions(-)


  git pull ssh://gerrit.cloudera.org:29418/kudu refs/changes/82/6482/4
-- 
To view, visit http://gerrit.cloudera.org:8080/6482
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings

Gerrit-MessageType: newpatchset
Gerrit-Change-Id: I6ab24681dfbb3b1e6f08d52eb0647a5f3ca6851f
Gerrit-PatchSet: 4
Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-Owner: Todd Lipcon <to...@apache.org>
Gerrit-Reviewer: David Ribeiro Alves <dr...@apache.org>
Gerrit-Reviewer: Kudu Jenkins