[kudu-CR] Add a RpcContext::RespondFailure() method

2016-07-12 Thread David Ribeiro Alves (Code Review)
David Ribeiro Alves has abandoned this change. Change subject: Add a RpcContext::RespondFailure() method .. Abandoned ok, abandoning this and opting for a RespondNoCache method. Will add it in the first patch where it's used.

[kudu-CR] Add a RpcContext::RespondFailure() method

2016-07-12 Thread Kudu Jenkins (Code Review)
Kudu Jenkins has posted comments on this change. Change subject: Add a RpcContext::RespondFailure() method .. Patch Set 21: Build Started http://104.196.14.100/job/kudu-gerrit/2337/ -- To view, visit

[kudu-CR] Add a RpcContext::RespondFailure() method

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

[kudu-CR] Add a RpcContext::RespondFailure() method

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

[kudu-CR] Add a RpcContext::RespondFailure() method

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/3191 to look at the new patch set (#16). Change subject: Add a RpcContext::RespondFailure() method .. Add a

[kudu-CR] Add a RpcContext::RespondFailure() method

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

[kudu-CR] Add a RpcContext::RespondFailure() method

2016-07-06 Thread Todd Lipcon (Code Review)
Todd Lipcon has posted comments on this change. Change subject: Add a RpcContext::RespondFailure() method .. Patch Set 15: (3 comments) http://gerrit.cloudera.org:8080/#/c/3191/15/src/kudu/rpc/rpc_context.h File

[kudu-CR] Add a RpcContext::RespondFailure() method

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

[kudu-CR] Add a RpcContext::RespondFailure() method

2016-06-30 Thread Kudu Jenkins (Code Review)
Kudu Jenkins has posted comments on this change. Change subject: Add a RpcContext::RespondFailure() method .. Patch Set 14: Build Started http://104.196.14.100/job/kudu-gerrit/2133/ -- To view, visit

[kudu-CR] Add a RpcContext::RespondFailure() method

2016-06-27 Thread Kudu Jenkins (Code Review)
Kudu Jenkins has posted comments on this change. Change subject: Add a RpcContext::RespondFailure() method .. Patch Set 13: -Verified Build Started http://104.196.14.100/job/kudu-gerrit/2023/ -- To view, visit

[kudu-CR] Add a RpcContext::RespondFailure() method

2016-06-27 Thread Kudu Jenkins (Code Review)
Kudu Jenkins has posted comments on this change. Change subject: Add a RpcContext::RespondFailure() method .. Patch Set 13: -Verified Build Started http://104.196.14.100/job/kudu-gerrit/2022/ -- To view, visit

[kudu-CR] Add a RpcContext::RespondFailure() method

2016-06-27 Thread Kudu Jenkins (Code Review)
Kudu Jenkins has posted comments on this change. Change subject: Add a RpcContext::RespondFailure() method .. Patch Set 13: -Verified Build Started http://104.196.14.100/job/kudu-gerrit/2015/ -- To view, visit

[kudu-CR] Add a RpcContext::RespondFailure() method

2016-06-22 Thread Kudu Jenkins (Code Review)
Kudu Jenkins has posted comments on this change. Change subject: Add a RpcContext::RespondFailure() method .. Patch Set 12: Build Started http://104.196.14.100/job/kudu-gerrit/1962/ -- To view, visit

[kudu-CR] Add a RpcContext::RespondFailure() method

2016-06-20 Thread David Ribeiro Alves (Code Review)
David Ribeiro Alves has posted comments on this change. Change subject: Add a RpcContext::RespondFailure() method .. Patch Set 10: Verified+1 unrelated flake MasterTest.TestMasterMetadataConsistentDespiteFailures -- To view,

[kudu-CR] Add a RpcContext::RespondFailure() method

2016-06-20 Thread Kudu Jenkins (Code Review)
Kudu Jenkins has posted comments on this change. Change subject: Add a RpcContext::RespondFailure() method .. Patch Set 10: Build Started http://104.196.14.100/job/kudu-gerrit/1903/ -- To view, visit

[kudu-CR] Add a RpcContext::RespondFailure() method

2016-06-20 Thread David Ribeiro Alves (Code Review)
Hello Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/3191 to look at the new patch set (#10). Change subject: Add a RpcContext::RespondFailure() method .. Add a

[kudu-CR] Add a RpcContext::RespondFailure() method

2016-06-20 Thread Kudu Jenkins (Code Review)
Kudu Jenkins has posted comments on this change. Change subject: Add a RpcContext::RespondFailure() method .. Patch Set 9: Build Started http://104.196.14.100/job/kudu-gerrit/1898/ -- To view, visit

[kudu-CR] Add a RpcContext::RespondFailure() method

2016-06-20 Thread Kudu Jenkins (Code Review)
Kudu Jenkins has posted comments on this change. Change subject: Add a RpcContext::RespondFailure() method .. Patch Set 8: Build Started http://104.196.14.100/job/kudu-gerrit/1886/ -- To view, visit

[kudu-CR] Add a RpcContext::RespondFailure() method

2016-06-20 Thread Kudu Jenkins (Code Review)
Kudu Jenkins has posted comments on this change. Change subject: Add a RpcContext::RespondFailure() method .. Patch Set 7: Build Started http://104.196.14.100/job/kudu-gerrit/1880/ -- To view, visit

[kudu-CR] Add a RpcContext::RespondFailure() method

2016-06-20 Thread David Ribeiro Alves (Code Review)
Hello Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/3191 to look at the new patch set (#7). Change subject: Add a RpcContext::RespondFailure() method .. Add a

[kudu-CR] Add a RpcContext::RespondFailure() method

2016-06-20 Thread David Ribeiro Alves (Code Review)
David Ribeiro Alves has posted comments on this change. Change subject: Add a RpcContext::RespondFailure() method .. Patch Set 6: (2 comments) bq. wonder whether it would be a clearer API to instead have a flag like

[kudu-CR] Add a RpcContext::RespondFailure() method

2016-05-30 Thread Todd Lipcon (Code Review)
Todd Lipcon has posted comments on this change. Change subject: Add a RpcContext::RespondFailure() method .. Patch Set 6: (2 comments) wonder whether it would be a clearer API to instead have a flag like

[kudu-CR] Add a RpcContext::RespondFailure() method

2016-05-24 Thread Adar Dembo (Code Review)
Adar Dembo has posted comments on this change. Change subject: Add a RpcContext::RespondFailure() method .. Patch Set 6: This patch is incredibly confusing stand alone. I think it's better served being part of the integration

[kudu-CR] Add a RpcContext::RespondFailure() method

2016-05-24 Thread David Ribeiro Alves (Code Review)
Hello Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/3191 to look at the new patch set (#6). Change subject: Add a RpcContext::RespondFailure() method .. Add a

[kudu-CR] Add a RpcContext::RespondFailure() method

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

[kudu-CR] Add a RpcContext::RespondFailure() method

2016-05-24 Thread David Ribeiro Alves (Code Review)
Hello Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/3191 to look at the new patch set (#2). Change subject: Add a RpcContext::RespondFailure() method .. Add a