Jean-Daniel Cryans has posted comments on this change.

Change subject: Blogpost describing predicate evaluation pushdown
......................................................................


Patch Set 2:

(5 comments)

Great stuff Andrew!

http://gerrit.cloudera.org:8080/#/c/4384/2/_posts/predicate-pushdown.md
File _posts/predicate-pushdown.md:

PS2, Line 7: with the Apache Kudu team this summer
Technically it was with the Apache Kudu team at Cloudera, else this reads like 
your internship was at the ASF :P


PS2, Line 10:  General Availability
You mean 1.0?


Line 14: 
I would add another paragraph that introduces your work, along with some perf 
numbers to tease people into reading more.


PS2, Line 68: scans
scan*


Line 126: 
You should add links to the commits related to this work.


-- 
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: 2
Gerrit-Project: kudu
Gerrit-Branch: gh-pages
Gerrit-Owner: Andrew Wong <anjuw...@g.ucla.edu>
Gerrit-Reviewer: Adar Dembo <a...@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