[kudu-CR] KUDU-1444. Get resource metrics of a scan.

2016-05-30 Thread zhen.zhang (Code Review)
Hello Todd Lipcon, Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/3013 to look at the new patch set (#9). Change subject: KUDU-1444. Get resource metrics of a scan. ..

[kudu-CR] KUDU-1444. Get resource metrics of a scan.

2016-05-30 Thread Jean-Daniel Cryans (Code Review)
Jean-Daniel Cryans has posted comments on this change. Change subject: KUDU-1444. Get resource metrics of a scan. .. Patch Set 8: > What should I do to help rebase? Try rebasing locally with your patch, see if you can solve

[kudu-CR] KUDU-1444. Get resource metrics of a scan.

2016-05-30 Thread zhen.zhang (Code Review)
zhen.zhang has posted comments on this change. Change subject: KUDU-1444. Get resource metrics of a scan. .. Patch Set 8: What should I do to help rebase? -- To view, visit http://gerrit.cloudera.org:8080/3013 To

[kudu-CR] KUDU-1444. Get resource metrics of a scan.

2016-05-30 Thread Todd Lipcon (Code Review)
Todd Lipcon has posted comments on this change. Change subject: KUDU-1444. Get resource metrics of a scan. .. Patch Set 8: Code-Review+2 -- To view, visit http://gerrit.cloudera.org:8080/3013 To unsubscribe, visit

[kudu-CR] KUDU-1444. Get resource metrics of a scan.

2016-05-30 Thread Todd Lipcon (Code Review)
Todd Lipcon has posted comments on this change. Change subject: KUDU-1444. Get resource metrics of a scan. .. Patch Set 8: Looks like this needs a manual rebase -- To view, visit http://gerrit.cloudera.org:8080/3013 To

[kudu-CR] KUDU-1444. Get resource metrics of a scan.

2016-05-26 Thread zhen.zhang (Code Review)
Hello Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/3013 to look at the new patch set (#8). Change subject: KUDU-1444. Get resource metrics of a scan. .. KUDU-1444. Get

[kudu-CR] KUDU-1444. Get resource metrics of a scan.

2016-05-26 Thread zhen.zhang (Code Review)
zhen.zhang has posted comments on this change. Change subject: KUDU-1444. Get resource metrics of a scan. .. Patch Set 7: (1 comment) http://gerrit.cloudera.org:8080/#/c/3013/7/src/kudu/client/client-test.cc File

[kudu-CR] KUDU-1444. Get resource metrics of a scan.

2016-05-26 Thread zhen.zhang (Code Review)
Hello Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/3013 to look at the new patch set (#7). Change subject: KUDU-1444. Get resource metrics of a scan. .. KUDU-1444. Get

[kudu-CR] KUDU-1444. Get resource metrics of a scan.

2016-05-26 Thread Todd Lipcon (Code Review)
Todd Lipcon has posted comments on this change. Change subject: KUDU-1444. Get resource metrics of a scan. .. Patch Set 6: OK. The code now looks good, but missing a test. Sorry, I should have mentioned that previously. Maybe

[kudu-CR] KUDU-1444. Get resource metrics of a scan.

2016-05-25 Thread Todd Lipcon (Code Review)
Todd Lipcon has posted comments on this change. Change subject: KUDU-1444. Get resource metrics of a scan. .. Patch Set 5: (7 comments) http://gerrit.cloudera.org:8080/#/c/3013/5/src/kudu/client/client.h File