[kudu-CR] Expose running maintenance op info

2017-07-28 Thread Sam Okrent (Code Review)
Hello Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/7537 to look at the new patch set (#2). Change subject: Expose running maintenance op info .. Expose running

[kudu-CR] Expose running maintenance op info

2017-07-28 Thread Sam Okrent (Code Review)
Sam Okrent has uploaded a new change for review. http://gerrit.cloudera.org:8080/7537 Change subject: Expose running maintenance op info .. Expose running maintenance op info Previously, the maintenance manager stored the

[kudu-CR] consensus: switch RaftConsensus to shared ptr

2017-07-28 Thread Todd Lipcon (Code Review)
Todd Lipcon has posted comments on this change. Change subject: consensus: switch RaftConsensus to shared_ptr .. Patch Set 2: Code-Review+2 -- To view, visit http://gerrit.cloudera.org:8080/7531 To unsubscribe, visit

[kudu-CR] consensus: switch RaftConsensus to shared ptr

2017-07-28 Thread Hao Hao (Code Review)
Hao Hao has posted comments on this change. Change subject: consensus: switch RaftConsensus to shared_ptr .. Patch Set 2: (1 comment) http://gerrit.cloudera.org:8080/#/c/7531/1/src/kudu/consensus/raft_consensus.h File

[kudu-CR] log block manager: mark container as read-only after syncing error

2017-07-28 Thread Hao Hao (Code Review)
Hao Hao has posted comments on this change. Change subject: log block manager: mark container as read-only after syncing error .. Patch Set 18: (2 comments)

[kudu-CR] log block manager: mark container as read-only after syncing error

2017-07-28 Thread Hao Hao (Code Review)
Hello Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/7374 to look at the new patch set (#18). Change subject: log block manager: mark container as read-only after syncing error

[kudu-CR] log block manager: mark container as read-only after syncing error

2017-07-28 Thread Adar Dembo (Code Review)
Adar Dembo has posted comments on this change. Change subject: log block manager: mark container as read-only after syncing error .. Patch Set 17: (2 comments)

[kudu-CR] consensus: switch RaftConsensus to shared ptr

2017-07-28 Thread Adar Dembo (Code Review)
Adar Dembo has posted comments on this change. Change subject: consensus: switch RaftConsensus to shared_ptr .. Patch Set 1: (2 comments) http://gerrit.cloudera.org:8080/#/c/7531/1/src/kudu/consensus/raft_consensus.cc File

[kudu-CR] consensus: switch RaftConsensus to shared ptr

2017-07-28 Thread Adar Dembo (Code Review)
Hello Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/7531 to look at the new patch set (#2). Change subject: consensus: switch RaftConsensus to shared_ptr .. consensus:

[kudu-CR] process memory::HardLimit() should init limits

2017-07-28 Thread Adar Dembo (Code Review)
Adar Dembo has posted comments on this change. Change subject: process_memory::HardLimit() should init limits .. Patch Set 1: Code-Review+2 -- To view, visit http://gerrit.cloudera.org:8080/7534 To unsubscribe, visit

[kudu-CR] process memory::HardLimit() should init limits

2017-07-28 Thread Adar Dembo (Code Review)
Adar Dembo has submitted this change and it was merged. Change subject: process_memory::HardLimit() should init limits .. process_memory::HardLimit() should init limits This fixes a TSAN race which showed up on the flaky

[kudu-CR] WIP: clock: add a built-in NTP client implementation

2017-07-28 Thread Todd Lipcon (Code Review)
Hello Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/7477 to look at the new patch set (#5). Change subject: WIP: clock: add a built-in NTP client implementation .. WIP:

[kudu-CR] clock: refactor interaction with NTP into a new interface

2017-07-28 Thread Todd Lipcon (Code Review)
Hello Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/7520 to look at the new patch set (#3). Change subject: clock: refactor interaction with NTP into a new interface ..

[kudu-CR] Move clock-related classes to src/kudu/clock

2017-07-28 Thread Todd Lipcon (Code Review)
Hello Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/7519 to look at the new patch set (#3). Change subject: Move clock-related classes to src/kudu/clock .. Move

[kudu-CR] [tools] Address potential flakiness of TestMoveTablet

2017-07-28 Thread Mike Percy (Code Review)
Mike Percy has posted comments on this change. Change subject: [tools] Address potential flakiness of TestMoveTablet .. Patch Set 1: (1 comment) http://gerrit.cloudera.org:8080/#/c/7533/1//COMMIT_MSG Commit Message: Line 14:

[kudu-CR] process memory::HardLimit() should init limits

2017-07-28 Thread Todd Lipcon (Code Review)
Hello Adar Dembo, I'd like you to do a code review. Please visit http://gerrit.cloudera.org:8080/7534 to review the following change. Change subject: process_memory::HardLimit() should init limits ..

[kudu-CR] [docs] Bump the data size and tablets limits

2017-07-28 Thread Todd Lipcon (Code Review)
Todd Lipcon has posted comments on this change. Change subject: [docs] Bump the data size and tablets limits .. Patch Set 2: Verified+1 -- To view, visit http://gerrit.cloudera.org:8080/7503 To unsubscribe, visit

[kudu-CR] [docs] Bump the data size and tablets limits

2017-07-28 Thread Todd Lipcon (Code Review)
Todd Lipcon has submitted this change and it was merged. Change subject: [docs] Bump the data size and tablets limits .. [docs] Bump the data size and tablets limits Some of the changes that landed in 1.4.0, namely Todd's

[kudu-CR] [docs] Bump the data size and tablets limits

2017-07-28 Thread Todd Lipcon (Code Review)
Todd Lipcon has posted comments on this change. Change subject: [docs] Bump the data size and tablets limits .. Patch Set 2: Code-Review+2 -- To view, visit http://gerrit.cloudera.org:8080/7503 To unsubscribe, visit

[kudu-CR] Fix a TSAN race on Webserver::path handlers

2017-07-28 Thread Todd Lipcon (Code Review)
Todd Lipcon has abandoned this change. Change subject: Fix a TSAN race on Webserver::path_handlers_ .. Abandoned this doesn't seem to show up on the dashboard anymore (I guess we don't mutate the path_handlers_ list after we

[kudu-CR] KUDU-969 (part 1): simplify tablet bootstrap-test

2017-07-28 Thread Todd Lipcon (Code Review)
Todd Lipcon has abandoned this change. Change subject: KUDU-969 (part 1): simplify tablet_bootstrap-test .. Abandoned stale, doesn't seem necessary anymore -- To view, visit http://gerrit.cloudera.org:8080/947 To

[kudu-CR] [tools] Address potential flakiness of TestMoveTablet

2017-07-28 Thread Todd Lipcon (Code Review)
Todd Lipcon has posted comments on this change. Change subject: [tools] Address potential flakiness of TestMoveTablet .. Patch Set 1: (2 comments) http://gerrit.cloudera.org:8080/#/c/7533/1//COMMIT_MSG Commit Message: Line

[kudu-CR] [tools] Address potential flakiness of TestMoveTablet

2017-07-28 Thread Will Berkeley (Code Review)
Will Berkeley has uploaded a new change for review. http://gerrit.cloudera.org:8080/7533 Change subject: [tools] Address potential flakiness of TestMoveTablet .. [tools] Address potential flakiness of TestMoveTablet When, by