[kudu-CR] rpc: some small cleanup in ConnectionId

2017-08-17 Thread Todd Lipcon (Code Review)
Todd Lipcon has submitted this change and it was merged. Change subject: rpc: some small cleanup in ConnectionId .. rpc: some small cleanup in ConnectionId Change-Id: I0788052f8c943ef102f3f551a85a8b219c65c361 Reviewed-on:

[kudu-CR] rpc: some small cleanup in ConnectionId

2017-08-16 Thread Alexey Serbin (Code Review)
Alexey Serbin has posted comments on this change. Change subject: rpc: some small cleanup in ConnectionId .. Patch Set 2: Code-Review+2 It seems there were unrelated flaky tests in RELEASE and SAN configs, but overall LGTM.

[kudu-CR] rpc: some small cleanup in ConnectionId

2017-08-16 Thread Todd Lipcon (Code Review)
Hello Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/7686 to look at the new patch set (#2). Change subject: rpc: some small cleanup in ConnectionId .. rpc: some small

[kudu-CR] rpc: some small cleanup in ConnectionId

2017-08-16 Thread Todd Lipcon (Code Review)
Todd Lipcon has posted comments on this change. Change subject: rpc: some small cleanup in ConnectionId .. Patch Set 1: (1 comment) http://gerrit.cloudera.org:8080/#/c/7686/1/src/kudu/rpc/proxy.cc File src/kudu/rpc/proxy.cc:

[kudu-CR] rpc: some small cleanup in ConnectionId

2017-08-16 Thread Alexey Serbin (Code Review)
Alexey Serbin has posted comments on this change. Change subject: rpc: some small cleanup in ConnectionId .. Patch Set 1: (1 comment) http://gerrit.cloudera.org:8080/#/c/7686/1/src/kudu/rpc/proxy.cc File

[kudu-CR] rpc: some small cleanup in ConnectionId

2017-08-15 Thread Todd Lipcon (Code Review)
Hello Sailesh Mukil, Alexey Serbin, I'd like you to do a code review. Please visit http://gerrit.cloudera.org:8080/7686 to review the following change. Change subject: rpc: some small cleanup in ConnectionId .. rpc: some