Dan Burkert has posted comments on this change.

Change subject: Create design docs directory and add partition pruning design 
doc
......................................................................


Patch Set 2:

(2 comments)

http://gerrit.cloudera.org:8080/#/c/2149/2/docs/design-docs/partition-pruning.md
File docs/design-docs/partition-pruning.md:

Line 105: pruning can occur. Pruned tablets never need to be contacted (though 
their
> It's not so much an interval tree (the data structure) as a list of interva
yah it doesn't necessarily need to be as fancy as an interval tree.  A 
vector<tuple<string, string>> representing a set of partition key bounds would 
probably do the trick.


Line 108: The server will go one step further by adding the server's primary 
key bounds to
> oh... yea, that's pretty contrived :) If the optimization here is basically
It should be pretty much free since we are already going to do the predicate 
normalization in case the client hasn't done it.


-- 
To view, visit http://gerrit.cloudera.org:8080/2149
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings

Gerrit-MessageType: comment
Gerrit-Change-Id: Ifacf3667f364286bf5a5f0c4931328035a9b517e
Gerrit-PatchSet: 2
Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-Owner: Dan Burkert <[email protected]>
Gerrit-Reviewer: Adar Dembo <[email protected]>
Gerrit-Reviewer: Dan Burkert <[email protected]>
Gerrit-Reviewer: David Ribeiro Alves <[email protected]>
Gerrit-Reviewer: Jean-Daniel Cryans
Gerrit-Reviewer: Kudu Jenkins
Gerrit-Reviewer: Mike Percy <[email protected]>
Gerrit-Reviewer: Misty Stanley-Jones <[email protected]>
Gerrit-Reviewer: Todd Lipcon <[email protected]>
Gerrit-HasComments: Yes

Reply via email to