[kudu-CR] tracing: gzip the trace JSON ajax response

2016-09-01 Thread Todd Lipcon (Code Review)
Todd Lipcon has posted comments on this change. Change subject: tracing: gzip the trace JSON ajax response .. Patch Set 3: (2 comments) http://gerrit.cloudera.org:8080/#/c/4154/3/src/kudu/server/tracing-path-handlers.cc File s

[kudu-CR] tracing: gzip the trace JSON ajax response

2016-09-01 Thread Alexey Serbin (Code Review)
Alexey Serbin has posted comments on this change. Change subject: tracing: gzip the trace JSON ajax response .. Patch Set 3: (3 comments) http://gerrit.cloudera.org:8080/#/c/4154/3/src/kudu/server/tracing-path-handlers.cc File

[kudu-CR] tracing: gzip the trace JSON ajax response

2016-09-01 Thread Todd Lipcon (Code Review)
Todd Lipcon has submitted this change and it was merged. Change subject: tracing: gzip the trace JSON ajax response .. tracing: gzip the trace JSON ajax response This enables gzipping of the actual trace which is sent back over

[kudu-CR] tracing: gzip the trace JSON ajax response

2016-09-01 Thread David Ribeiro Alves (Code Review)
David Ribeiro Alves has posted comments on this change. Change subject: tracing: gzip the trace JSON ajax response .. Patch Set 2: Code-Review+2 -- To view, visit http://gerrit.cloudera.org:8080/4154 To unsubscribe, visit http

[kudu-CR] tracing: gzip the trace JSON ajax response

2016-08-31 Thread Todd Lipcon (Code Review)
Hello Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/4154 to look at the new patch set (#2). Change subject: tracing: gzip the trace JSON ajax response .. tracing: gzip the

[kudu-CR] tracing: gzip the trace JSON ajax response

2016-08-31 Thread Kudu Jenkins (Code Review)
Kudu Jenkins has posted comments on this change. Change subject: tracing: gzip the trace JSON ajax response .. Patch Set 2: Build Started http://104.196.14.100/job/kudu-gerrit/3181/ -- To view, visit http://gerrit.cloudera.or

[kudu-CR] tracing: gzip the trace JSON ajax response

2016-08-31 Thread Todd Lipcon (Code Review)
Todd Lipcon has posted comments on this change. Change subject: tracing: gzip the trace JSON ajax response .. Patch Set 1: (3 comments) http://gerrit.cloudera.org:8080/#/c/4154/1//COMMIT_MSG Commit Message: Line 18: version u

[kudu-CR] tracing: gzip the trace JSON ajax response

2016-08-31 Thread Todd Lipcon (Code Review)
Todd Lipcon has posted comments on this change. Change subject: tracing: gzip the trace JSON ajax response .. Patch Set 1: (3 comments) http://gerrit.cloudera.org:8080/#/c/4154/1//COMMIT_MSG Commit Message: Line 16: the serve

[kudu-CR] tracing: gzip the trace JSON ajax response

2016-08-31 Thread David Ribeiro Alves (Code Review)
David Ribeiro Alves has posted comments on this change. Change subject: tracing: gzip the trace JSON ajax response .. Patch Set 1: (5 comments) http://gerrit.cloudera.org:8080/#/c/4154/1//COMMIT_MSG Commit Message: Line 16: t

[kudu-CR] tracing: gzip the trace JSON ajax response

2016-08-28 Thread Kudu Jenkins (Code Review)
Kudu Jenkins has posted comments on this change. Change subject: tracing: gzip the trace JSON ajax response .. Patch Set 1: Build Started http://104.196.14.100/job/kudu-gerrit/3125/ -- To view, visit http://gerrit.cloudera.or

[kudu-CR] tracing: gzip the trace JSON ajax response

2016-08-28 Thread Todd Lipcon (Code Review)
Hello Mike Percy, I'd like you to do a code review. Please visit http://gerrit.cloudera.org:8080/4154 to review the following change. Change subject: tracing: gzip the trace JSON ajax response .. tracing: gzip the trace J