[kudu-CR] Integrate Mustache templates to webserver

2017-07-26 Thread Dan Burkert (Code Review)
Dan Burkert has submitted this change and it was merged. Change subject: Integrate Mustache templates to webserver .. Integrate Mustache templates to webserver This commit alters the design of webserver path handlers to use Mus

[kudu-CR] Integrate Mustache templates to webserver

2017-07-26 Thread Dan Burkert (Code Review)
Dan Burkert has posted comments on this change. Change subject: Integrate Mustache templates to webserver .. Patch Set 14: Code-Review+2 -- To view, visit http://gerrit.cloudera.org:8080/7448 To unsubscribe, visit http://gerri

[kudu-CR] Integrate Mustache templates to webserver

2017-07-26 Thread Sam Okrent (Code Review)
Hello Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/7448 to look at the new patch set (#14). Change subject: Integrate Mustache templates to webserver .. Integrate Mustache

[kudu-CR] Integrate Mustache templates to webserver

2017-07-25 Thread Dan Burkert (Code Review)
Dan Burkert has posted comments on this change. Change subject: Integrate Mustache templates to webserver .. Patch Set 13: (1 comment) http://gerrit.cloudera.org:8080/#/c/7448/13/src/kudu/util/easy_json.h File src/kudu/util/ea

[kudu-CR] Integrate Mustache templates to webserver

2017-07-25 Thread Sam Okrent (Code Review)
Sam Okrent has posted comments on this change. Change subject: Integrate Mustache templates to webserver .. Patch Set 12: (1 comment) http://gerrit.cloudera.org:8080/#/c/7448/12/src/kudu/server/webserver.cc File src/kudu/serve

[kudu-CR] Integrate Mustache templates to webserver

2017-07-25 Thread Sam Okrent (Code Review)
Hello Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/7448 to look at the new patch set (#13). Change subject: Integrate Mustache templates to webserver .. Integrate Mustache

[kudu-CR] Integrate Mustache templates to webserver

2017-07-25 Thread Dan Burkert (Code Review)
Dan Burkert has posted comments on this change. Change subject: Integrate Mustache templates to webserver .. Patch Set 12: (1 comment) http://gerrit.cloudera.org:8080/#/c/7448/12/src/kudu/server/webserver.cc File src/kudu/serv

[kudu-CR] Integrate Mustache templates to webserver

2017-07-24 Thread Sam Okrent (Code Review)
Hello Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/7448 to look at the new patch set (#12). Change subject: Integrate Mustache templates to webserver .. Integrate Mustache

[kudu-CR] Integrate Mustache templates to webserver

2017-07-24 Thread Sam Okrent (Code Review)
Hello Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/7448 to look at the new patch set (#11). Change subject: Integrate Mustache templates to webserver .. Integrate Mustache

[kudu-CR] Integrate Mustache templates to webserver

2017-07-24 Thread Dan Burkert (Code Review)
Dan Burkert has posted comments on this change. Change subject: Integrate Mustache templates to webserver .. Patch Set 9: (1 comment) http://gerrit.cloudera.org:8080/#/c/7448/9/src/kudu/server/webserver.cc File src/kudu/server

[kudu-CR] Integrate Mustache templates to webserver

2017-07-24 Thread Sam Okrent (Code Review)
Sam Okrent has posted comments on this change. Change subject: Integrate Mustache templates to webserver .. Patch Set 10: (6 comments) http://gerrit.cloudera.org:8080/#/c/7448/9/src/kudu/server/webserver.cc File src/kudu/serve

[kudu-CR] Integrate Mustache templates to webserver

2017-07-24 Thread Sam Okrent (Code Review)
Hello Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/7448 to look at the new patch set (#10). Change subject: Integrate Mustache templates to webserver .. Integrate Mustache

[kudu-CR] Integrate Mustache templates to webserver

2017-07-24 Thread Dan Burkert (Code Review)
Dan Burkert has posted comments on this change. Change subject: Integrate Mustache templates to webserver .. Patch Set 9: (6 comments) http://gerrit.cloudera.org:8080/#/c/7448/9/src/kudu/server/webserver.cc File src/kudu/serve

[kudu-CR] Integrate Mustache templates to webserver

2017-07-19 Thread Sam Okrent (Code Review)
Sam Okrent has posted comments on this change. Change subject: Integrate Mustache templates to webserver .. Patch Set 9: This commit didn't go anywhere near raft consensus, and the log doesn't look like the failure of RaftCons

[kudu-CR] Integrate Mustache templates to webserver

2017-07-19 Thread Sam Okrent (Code Review)
Hello Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/7448 to look at the new patch set (#9). Change subject: Integrate Mustache templates to webserver .. Integrate Mustache

[kudu-CR] Integrate Mustache templates to webserver

2017-07-19 Thread Dan Burkert (Code Review)
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 sti

[kudu-CR] Integrate Mustache templates to webserver

2017-07-19 Thread Sam Okrent (Code Review)
Hello Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/7448 to look at the new patch set (#8). Change subject: Integrate Mustache templates to webserver .. Integrate Mustache

[kudu-CR] Integrate Mustache templates to webserver

2017-07-19 Thread Sam Okrent (Code Review)
Hello Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/7448 to look at the new patch set (#7). Change subject: Integrate Mustache templates to webserver .. Integrate Mustache

[kudu-CR] Integrate Mustache templates to webserver

2017-07-19 Thread Sam Okrent (Code Review)
Hello Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/7448 to look at the new patch set (#6). Change subject: Integrate Mustache templates to webserver .. Integrate Mustache

[kudu-CR] Integrate Mustache templates to webserver

2017-07-17 Thread Sam Okrent (Code Review)
Hello Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/7448 to look at the new patch set (#5). Change subject: Integrate Mustache templates to webserver .. Integrate Mustache

[kudu-CR] Integrate Mustache templates to webserver

2017-07-17 Thread Sam Okrent (Code Review)
Hello Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/7448 to look at the new patch set (#4). Change subject: Integrate Mustache templates to webserver .. Integrate Mustache

[kudu-CR] Integrate Mustache templates to webserver

2017-07-17 Thread Sam Okrent (Code Review)
Hello Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/7448 to look at the new patch set (#3). Change subject: Integrate Mustache templates to webserver .. Integrate Mustache

[kudu-CR] Integrate Mustache templates to webserver

2017-07-17 Thread Sam Okrent (Code Review)
Hello Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/7448 to look at the new patch set (#2). Change subject: Integrate Mustache templates to webserver .. Integrate Mustache

[kudu-CR] Integrate Mustache templates to webserver

2017-07-17 Thread Sam Okrent (Code Review)
Sam Okrent has uploaded a new change for review. http://gerrit.cloudera.org:8080/7448 Change subject: Integrate Mustache templates to webserver .. Integrate Mustache templates to webserver This commit alters the design of web