Dan Burkert has posted comments on this change.

Change subject: Integrate Mustache templates to webserver
......................................................................


Patch Set 8:

(1 comment)

sorry I realized I had this old review item from a previous revision, but looks 
like it still applies.  Will review the latest revision shortly.

http://gerrit.cloudera.org:8080/#/c/7448/5/src/kudu/server/CMakeLists.txt
File src/kudu/server/CMakeLists.txt:

Line 94:   mustache)
keep in alphabetical order


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

Gerrit-MessageType: comment
Gerrit-Change-Id: I8cf8181b2b2904a57c9b24ce73316b92a4f6700f
Gerrit-PatchSet: 8
Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-Owner: Sam Okrent <[email protected]>
Gerrit-Reviewer: Dan Burkert <[email protected]>
Gerrit-Reviewer: Kudu Jenkins
Gerrit-Reviewer: Tidy Bot
Gerrit-HasComments: Yes

Reply via email to