[kudu-CR] [rpc] WIP: Introduce configurable options to Messenger

2017-03-30 Thread Sailesh Mukil (Code Review)
Sailesh Mukil has posted comments on this change. Change subject: [rpc] WIP: Introduce configurable options to Messenger .. Patch Set 1: > Build Failed > > http://104.196.14.100/job/kudu-gerrit/7167/ : FAILURE I'll do the

[kudu-CR](branch-1.3.x) KUDU-1962. Fix NPE when master returns SERVER TOO BUSY

2017-03-30 Thread Jean-Daniel Cryans (Code Review)
Jean-Daniel Cryans has posted comments on this change. Change subject: KUDU-1962. Fix NPE when master returns SERVER_TOO_BUSY .. Patch Set 1: Code-Review+2 -- To view, visit http://gerrit.cloudera.org:8080/6519 To

[kudu-CR] tablet: check for row presence in rowset-wise order

2017-03-30 Thread David Ribeiro Alves (Code Review)
David Ribeiro Alves has posted comments on this change. Change subject: tablet: check for row presence in rowset-wise order .. Patch Set 8: (14 comments) http://gerrit.cloudera.org:8080/#/c/6483/8/src/kudu/tablet/row_op.h

[kudu-CR] KUDU-1875: Refuse unauthenticated connections from publicly routable IP addrs

2017-03-30 Thread Todd Lipcon (Code Review)
Todd Lipcon has posted comments on this change. Change subject: KUDU-1875: Refuse unauthenticated connections from publicly routable IP addrs .. Patch Set 1: (1 comment)

[kudu-CR] KUDU-1962. Fix NPE when master returns SERVER TOO BUSY

2017-03-30 Thread Jean-Daniel Cryans (Code Review)
Jean-Daniel Cryans has posted comments on this change. Change subject: KUDU-1962. Fix NPE when master returns SERVER_TOO_BUSY .. Patch Set 4: Code-Review+2 (1 comment)

[kudu-CR] KUDU-1962. Fix NPE when master returns SERVER TOO BUSY

2017-03-30 Thread Todd Lipcon (Code Review)
Hello Jean-Daniel Cryans, Matthew Jacobs, Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/6516 to look at the new patch set (#4). Change subject: KUDU-1962. Fix NPE when master returns SERVER_TOO_BUSY

[kudu-CR] KUDU-1962. Fix NPE when master returns SERVER TOO BUSY

2017-03-30 Thread Todd Lipcon (Code Review)
Hello Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/6516 to look at the new patch set (#3). Change subject: KUDU-1962. Fix NPE when master returns SERVER_TOO_BUSY ..

[kudu-CR] KUDU-1962. Fix NPE when master returns SERVER TOO BUSY

2017-03-30 Thread Todd Lipcon (Code Review)
Todd Lipcon has posted comments on this change. Change subject: KUDU-1962. Fix NPE when master returns SERVER_TOO_BUSY .. Patch Set 1: (3 comments)

[kudu-CR] KUDU-1962. Fix NPE when master returns SERVER TOO BUSY

2017-03-30 Thread Todd Lipcon (Code Review)
Hello Jean-Daniel Cryans, Matthew Jacobs, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/6516 to look at the new patch set (#2). Change subject: KUDU-1962. Fix NPE when master returns SERVER_TOO_BUSY

[kudu-CR] KUDU-1962. Fix NPE when master returns SERVER TOO BUSY

2017-03-30 Thread Todd Lipcon (Code Review)
Todd Lipcon has posted comments on this change. Change subject: KUDU-1962. Fix NPE when master returns SERVER_TOO_BUSY .. Patch Set 1: (1 comment)

[kudu-CR] KUDU-1962. Fix NPE when master returns SERVER TOO BUSY

2017-03-30 Thread Todd Lipcon (Code Review)
Todd Lipcon has posted comments on this change. Change subject: KUDU-1962. Fix NPE when master returns SERVER_TOO_BUSY .. Patch Set 1: (1 comment)

[kudu-CR] KUDU-1962. Fix NPE when master returns SERVER TOO BUSY

2017-03-30 Thread Todd Lipcon (Code Review)
Hello Jean-Daniel Cryans, Matthew Jacobs, I'd like you to do a code review. Please visit http://gerrit.cloudera.org:8080/6516 to review the following change. Change subject: KUDU-1962. Fix NPE when master returns SERVER_TOO_BUSY

[kudu-CR] KUDU-1875: Refuse unauthenticated connections from publicly routable IP addrs

2017-03-30 Thread Hao Hao (Code Review)
Hao Hao has uploaded a new change for review. http://gerrit.cloudera.org:8080/6514 Change subject: KUDU-1875: Refuse unauthenticated connections from publicly routable IP addrs .. KUDU-1875: Refuse unauthenticated