Alexey Serbin has posted comments on this change. Change subject: Blogpost describing predicate evaluation pushdown ......................................................................
Patch Set 4: (2 comments) http://gerrit.cloudera.org:8080/#/c/4384/4/_posts/predicate-pushdown.md File _posts/predicate-pushdown.md: PS4, Line 82: when there are too many unique values to fit in a dictionary, performance : suffers > Dictionary blocks have two different blocks: a vocabulary block that contai Thank you for the explanation. So, the talk here is about building and maintaining that dictionary, but not using it for scanning of plain-encoded blocks? PS4, Line 144: to 1.0 > version 1.0? 1.0 release? I would just drop 'to 1.0', but if the original intention was to emphasize the release quality of the project, then it's better to re-phrase the sentence. In the latter case, the exact structure of the would-be-sentence escapes me, but I hope you can come up with something. If you think it's OK as is, then leave it :) -- To view, visit http://gerrit.cloudera.org:8080/4384 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-MessageType: comment Gerrit-Change-Id: Ia3b593959194a6ce9190f562339dc04a1d8fceba Gerrit-PatchSet: 4 Gerrit-Project: kudu Gerrit-Branch: gh-pages Gerrit-Owner: Andrew Wong <anjuw...@g.ucla.edu> Gerrit-Reviewer: Adar Dembo <a...@cloudera.com> Gerrit-Reviewer: Alexey Serbin <aser...@cloudera.com> Gerrit-Reviewer: Andrew Wong <anjuw...@g.ucla.edu> Gerrit-Reviewer: Dan Burkert <d...@cloudera.com> Gerrit-Reviewer: Jean-Daniel Cryans <jdcry...@apache.org> Gerrit-Reviewer: Todd Lipcon <t...@apache.org> Gerrit-HasComments: Yes