[kudu-CR] Add a ToString() method to Proxy

2016-07-12 Thread Todd Lipcon (Code Review)
Todd Lipcon has posted comments on this change. Change subject: Add a ToString() method to Proxy .. Patch Set 9: Code-Review+2 -- To view, visit http://gerrit.cloudera.org:8080/3502 To unsubscribe, visit

[kudu-CR] Add a ToString() method to Proxy

2016-07-12 Thread Todd Lipcon (Code Review)
Todd Lipcon has submitted this change and it was merged. Change subject: Add a ToString() method to Proxy .. Add a ToString() method to Proxy ReplicatedRpc takes the server proxy type as a template argument and uses its

[kudu-CR] Add a ToString() method to Proxy

2016-07-08 Thread Kudu Jenkins (Code Review)
Kudu Jenkins has posted comments on this change. Change subject: Add a ToString() method to Proxy .. Patch Set 8: Build Started http://104.196.14.100/job/kudu-gerrit/2279/ -- To view, visit

[kudu-CR] Add a ToString() method to Proxy

2016-07-08 Thread Kudu Jenkins (Code Review)
Kudu Jenkins has posted comments on this change. Change subject: Add a ToString() method to Proxy .. Patch Set 7: Build Started http://104.196.14.100/job/kudu-gerrit/2259/ -- To view, visit

[kudu-CR] Add a ToString() method to Proxy

2016-07-08 Thread David Ribeiro Alves (Code Review)
David Ribeiro Alves has posted comments on this change. Change subject: Add a ToString() method to Proxy .. Patch Set 1: (1 comment) http://gerrit.cloudera.org:8080/#/c/3502/1/src/kudu/rpc/proxy.cc File src/kudu/rpc/proxy.cc:

[kudu-CR] Add a ToString() method to Proxy

2016-07-08 Thread Kudu Jenkins (Code Review)
Kudu Jenkins has posted comments on this change. Change subject: Add a ToString() method to Proxy .. Patch Set 6: Build Started http://104.196.14.100/job/kudu-gerrit/2249/ -- To view, visit

[kudu-CR] Add a ToString() method to Proxy

2016-07-08 Thread Kudu Jenkins (Code Review)
Kudu Jenkins has posted comments on this change. Change subject: Add a ToString() method to Proxy .. Patch Set 5: Build Started http://104.196.14.100/job/kudu-gerrit/2242/ -- To view, visit

[kudu-CR] Add a ToString() method to Proxy

2016-07-08 Thread David Ribeiro Alves (Code Review)
Hello Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/3502 to look at the new patch set (#4). Change subject: Add a ToString() method to Proxy .. Add a ToString() method to

[kudu-CR] Add a ToString() method to Proxy

2016-07-06 Thread Todd Lipcon (Code Review)
Todd Lipcon has posted comments on this change. Change subject: Add a ToString() method to Proxy .. Patch Set 3: see note above about adding an assertion -- To view, visit http://gerrit.cloudera.org:8080/3502 To unsubscribe,

[kudu-CR] Add a ToString() method to Proxy

2016-07-05 Thread Kudu Jenkins (Code Review)
Kudu Jenkins has posted comments on this change. Change subject: Add a ToString() method to Proxy .. Patch Set 3: Build Started http://104.196.14.100/job/kudu-gerrit/2165/ -- To view, visit

[kudu-CR] Add a ToString() method to Proxy

2016-06-27 Thread Todd Lipcon (Code Review)
Todd Lipcon has posted comments on this change. Change subject: Add a ToString() method to Proxy .. Patch Set 1: would be nice to add a new assertion to one of the RPC layer tests which calls this -- To view, visit

[kudu-CR] Add a ToString() method to Proxy

2016-06-27 Thread Adar Dembo (Code Review)
Adar Dembo has posted comments on this change. Change subject: Add a ToString() method to Proxy .. Patch Set 1: (1 comment) http://gerrit.cloudera.org:8080/#/c/3502/1/src/kudu/rpc/proxy.cc File src/kudu/rpc/proxy.cc: Line