Andrew Wong has posted comments on this change.

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


Patch Set 2:

(5 comments)

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 l
Done


PS2, Line 10:  General Availability
> You mean 1.0?
Done. Is just 1.0 fine? Or should it be version 1.0/v1.0? Is it widely 
understood what 1.0 is referring to?


Line 14: 
> I would add another paragraph that introduces your work, along with some pe
Will do. Wasn't sure how to bridge the header to here since I thought it might 
be redundant, but I agree, need to entice people more with those perf numbers.


PS2, Line 68: scans
> scan*
Done


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


-- 
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