[kudu-CR] KUDU-1866: Add request-side sidecars

2017-03-07 Thread Todd Lipcon (Code Review)
Todd Lipcon has submitted this change and it was merged. Change subject: KUDU-1866: Add request-side sidecars .. KUDU-1866: Add request-side sidecars This patch adds sidecars to client requests. Using the same mechanism as on

[kudu-CR] KUDU-1866: Add request-side sidecars

2017-03-07 Thread Todd Lipcon (Code Review)
Todd Lipcon has posted comments on this change. Change subject: KUDU-1866: Add request-side sidecars .. Patch Set 11: Code-Review+2 (1 comment) http://gerrit.cloudera.org:8080/#/c/5908/8/src/kudu/rpc/rpc_sidecar.cc File

[kudu-CR] KUDU-1866: Add request-side sidecars

2017-03-07 Thread Henry Robinson (Code Review)
Henry Robinson has posted comments on this change. Change subject: KUDU-1866: Add request-side sidecars .. Patch Set 11: Reverted to 32-bit offsets, and rebased to fix a merge conflict. -- To view, visit

[kudu-CR] KUDU-1866: Add request-side sidecars

2017-03-07 Thread Henry Robinson (Code Review)
Hello Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/5908 to look at the new patch set (#11). Change subject: KUDU-1866: Add request-side sidecars .. KUDU-1866: Add

[kudu-CR] KUDU-1866: Add request-side sidecars

2017-03-07 Thread Henry Robinson (Code Review)
Hello Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/5908 to look at the new patch set (#10). Change subject: KUDU-1866: Add request-side sidecars .. KUDU-1866: Add

[kudu-CR] KUDU-1866: Add request-side sidecars

2017-03-06 Thread Henry Robinson (Code Review)
Henry Robinson has posted comments on this change. Change subject: KUDU-1866: Add request-side sidecars .. Patch Set 8: Test failures due to Java client incompatibility. Will have to rethink (and probably revert) use of

[kudu-CR] KUDU-1866: Add request-side sidecars

2017-03-06 Thread Henry Robinson (Code Review)
Henry Robinson has posted comments on this change. Change subject: KUDU-1866: Add request-side sidecars .. Patch Set 8: (13 comments) Thanks for the review!

[kudu-CR] KUDU-1866: Add request-side sidecars

2017-03-06 Thread Henry Robinson (Code Review)
Hello Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/5908 to look at the new patch set (#9). Change subject: KUDU-1866: Add request-side sidecars .. KUDU-1866: Add

[kudu-CR] KUDU-1866: Add request-side sidecars

2017-03-06 Thread Todd Lipcon (Code Review)
Todd Lipcon has posted comments on this change. Change subject: KUDU-1866: Add request-side sidecars .. Patch Set 8: (13 comments) http://gerrit.cloudera.org:8080/#/c/5908/8/src/kudu/rpc/rpc_controller.cc File

[kudu-CR] KUDU-1866: Add request-side sidecars

2017-02-14 Thread Henry Robinson (Code Review)
Henry Robinson has posted comments on this change. Change subject: KUDU-1866: Add request-side sidecars .. Patch Set 7: (4 comments) http://gerrit.cloudera.org:8080/#/c/5908/7/src/kudu/client/scanner-internal.cc File

[kudu-CR] KUDU-1866: Add request-side sidecars

2017-02-14 Thread Henry Robinson (Code Review)
Hello Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/5908 to look at the new patch set (#8). Change subject: KUDU-1866: Add request-side sidecars .. KUDU-1866: Add

[kudu-CR] KUDU-1866: Add request-side sidecars

2017-02-10 Thread Henry Robinson (Code Review)
Hello Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/5908 to look at the new patch set (#7). Change subject: KUDU-1866: Add request-side sidecars .. KUDU-1866: Add

[kudu-CR] KUDU-1866: Add request-side sidecars

2017-02-10 Thread Henry Robinson (Code Review)
Henry Robinson has posted comments on this change. Change subject: KUDU-1866: Add request-side sidecars .. Patch Set 6: (1 comment) http://gerrit.cloudera.org:8080/#/c/5908/6/src/kudu/rpc/outbound_call.h File

[kudu-CR] KUDU-1866: Add request-side sidecars

2017-02-10 Thread Henry Robinson (Code Review)
Henry Robinson has posted comments on this change. Change subject: KUDU-1866: Add request-side sidecars .. Patch Set 6: Looks like an unrelated test failure to me, although I'm unable to build locally to confirm. -- To

[kudu-CR] KUDU-1866: Add request-side sidecars

2017-02-09 Thread Henry Robinson (Code Review)
Henry Robinson has posted comments on this change. Change subject: KUDU-1866: Add request-side sidecars .. Patch Set 6: Rebased to include boost changes. -- To view, visit http://gerrit.cloudera.org:8080/5908 To unsubscribe,

[kudu-CR] KUDU-1866: Add request-side sidecars

2017-02-09 Thread Henry Robinson (Code Review)
Hello Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/5908 to look at the new patch set (#5). Change subject: KUDU-1866: Add request-side sidecars .. KUDU-1866: Add

[kudu-CR] KUDU-1866: Add request-side sidecars

2017-02-09 Thread Henry Robinson (Code Review)
Henry Robinson has posted comments on this change. Change subject: KUDU-1866: Add request-side sidecars .. Patch Set 4: (20 comments) http://gerrit.cloudera.org:8080/#/c/5908/4//COMMIT_MSG Commit Message: Line 7: KUDU-1866: