[kudu-CR] [rpc] reopen outbound connections mode for reactor

2017-04-21 Thread Alexey Serbin (Code Review)
Alexey Serbin has uploaded a new patch set (#2). Change subject: [rpc] reopen_outbound_connections mode for reactor .. [rpc] reopen_outbound_connections mode for reactor Introduced a test mode for the RPC ReactorThread: reopen

[kudu-CR] [rpc] reopen outbound connections mode for reactor

2017-04-21 Thread Alexey Serbin (Code Review)
Alexey Serbin has uploaded a new change for review. http://gerrit.cloudera.org:8080/6710 Change subject: [rpc] reopen_outbound_connections mode for reactor .. [rpc] reopen_outbound_connections mode for reactor Introduced a

[kudu-CR] Add 'kudu master list' tool

2017-04-21 Thread Dan Burkert (Code Review)
Dan Burkert has posted comments on this change. Change subject: Add 'kudu master list' tool .. Patch Set 2: (1 comment) http://gerrit.cloudera.org:8080/#/c/6705/2/src/kudu/tools/tool_action_common.cc File

[kudu-CR] [rpc] handling ERROR UNAVAILABLE RPC error

2017-04-21 Thread Alexey Serbin (Code Review)
Alexey Serbin has posted comments on this change. Change subject: [rpc] handling ERROR_UNAVAILABLE RPC error .. Patch Set 6: (3 comments)

[kudu-CR] Tweak ListMasters RPC error handling

2017-04-21 Thread Adar Dembo (Code Review)
Adar Dembo has posted comments on this change. Change subject: Tweak ListMasters RPC error handling .. Patch Set 2: Code-Review+2 -- To view, visit http://gerrit.cloudera.org:8080/6704 To unsubscribe, visit

[kudu-CR] Tweak ListMasters RPC error handling

2017-04-21 Thread Dan Burkert (Code Review)
Dan Burkert has submitted this change and it was merged. Change subject: Tweak ListMasters RPC error handling .. Tweak ListMasters RPC error handling Unlike other master-specific RPCs, the ListMasters response contains an

[kudu-CR] Add 'kudu master list' tool

2017-04-21 Thread Dan Burkert (Code Review)
Dan Burkert has submitted this change and it was merged. Change subject: Add 'kudu master list' tool .. Add 'kudu master list' tool This also sneaks in a couple other changes: - kudu tserver list now has a 'heartbeat' column

[kudu-CR] KUDU-1952 Remove round-robin for block placement

2017-04-21 Thread Andrew Wong (Code Review)
Hello Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/6636 to look at the new patch set (#7). Change subject: KUDU-1952 Remove round-robin for block placement .. KUDU-1952

[kudu-CR] [rpc] handling ERROR UNAVAILABLE RPC error

2017-04-21 Thread Alexey Serbin (Code Review)
Hello Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/6640 to look at the new patch set (#7). Change subject: [rpc] handling ERROR_UNAVAILABLE RPC error .. [rpc] handling

[kudu-CR] Add 'kudu master list' tool

2017-04-21 Thread Adar Dembo (Code Review)
Adar Dembo has posted comments on this change. Change subject: Add 'kudu master list' tool .. Patch Set 2: Code-Review+2 (1 comment) http://gerrit.cloudera.org:8080/#/c/6705/2/src/kudu/tools/tool_action_common.cc File

[kudu-CR] [c++-client] clear non-covered entries from meta cache on table open

2017-04-21 Thread Dan Burkert (Code Review)
Hello Matthew Jacobs, Thomas Tauber-Marshall, Adar Dembo, Todd Lipcon, I'd like you to do a code review. Please visit http://gerrit.cloudera.org:8080/6713 to review the following change. Change subject: [c++-client] clear non-covered entries from meta cache on table open

[kudu-CR] KUDU-1952 Remove round-robin for block placement

2017-04-21 Thread Andrew Wong (Code Review)
Hello Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/6636 to look at the new patch set (#9). Change subject: KUDU-1952 Remove round-robin for block placement .. KUDU-1952

[kudu-CR] Add 'kudu master list' tool

2017-04-21 Thread Dan Burkert (Code Review)
Hello Adar Dembo, Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/6705 to look at the new patch set (#2). Change subject: Add 'kudu master list' tool .. Add 'kudu master

[kudu-CR] KUDU-1952 Remove round-robin for block placement

2017-04-21 Thread Andrew Wong (Code Review)
Hello Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/6636 to look at the new patch set (#8). Change subject: KUDU-1952 Remove round-robin for block placement .. KUDU-1952

[kudu-CR] [c++ client] re-acquire authn token if expired

2017-04-21 Thread Alexey Serbin (Code Review)
Hello Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/6648 to look at the new patch set (#4). Change subject: [c++ client] re-acquire authn token if expired .. [c++ client]

[kudu-CR] [rpc] handling ERROR UNAVAILABLE RPC error

2017-04-21 Thread Alexey Serbin (Code Review)
Hello Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/6640 to look at the new patch set (#9). Change subject: [rpc] handling ERROR_UNAVAILABLE RPC error .. [rpc] handling

[kudu-CR] [c++-client] clear non-covered entries from meta cache on table open

2017-04-21 Thread Adar Dembo (Code Review)
Adar Dembo has posted comments on this change. Change subject: [c++-client] clear non-covered entries from meta cache on table open .. Patch Set 1: (1 comment) I'll pose the same question I posed on Slack: can we conceive of

[kudu-CR] WIP [rpc] propagate RPC status for negotiation errors

2017-04-21 Thread Alexey Serbin (Code Review)
Alexey Serbin has abandoned this change. Change subject: WIP [rpc] propagate RPC status for negotiation errors .. Abandoned This change is now part of https://gerrit.cloudera.org/#/c/6648/ -- To view, visit

[kudu-CR] log block manager: corruptor test utility

2017-04-21 Thread Adar Dembo (Code Review)
Adar Dembo has posted comments on this change. Change subject: log block manager: corruptor test utility .. Patch Set 7: (2 comments) http://gerrit.cloudera.org:8080/#/c/6582/7/src/kudu/fs/log_block_manager-test-util.cc File

[kudu-CR] [rpc] handling ERROR UNAVAILABLE RPC error

2017-04-21 Thread Alexey Serbin (Code Review)
Hello Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/6640 to look at the new patch set (#8). Change subject: [rpc] handling ERROR_UNAVAILABLE RPC error .. [rpc] handling

[kudu-CR] WIP [client] retry RPC if authn token expires

2017-04-21 Thread Alexey Serbin (Code Review)
Alexey Serbin has abandoned this change. Change subject: WIP [client] retry RPC if authn token expires .. Abandoned This change has evolved into non-WIP patch: https://gerrit.cloudera.org/#/c/6648/ -- To view, visit