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

Reply via email to