[kudu-CR] external mini cluster: allow more flags on restart

2017-07-07 Thread Andrew Wong (Code Review)
Andrew Wong has abandoned this change. Change subject: external_mini_cluster: allow more flags on restart .. Abandoned Not needed -- To view, visit http://gerrit.cloudera.org:8080/7242 To unsubscribe, visit

[kudu-CR] external mini cluster: allow more flags on restart

2017-07-07 Thread Andrew Wong (Code Review)
Andrew Wong has posted comments on this change. Change subject: external_mini_cluster: allow more flags on restart .. Patch Set 4: (1 comment)

[kudu-CR] WIP disk failure: add persistent disk states

2017-07-07 Thread Andrew Wong (Code Review)
Hello Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/7270 to look at the new patch set (#3). Change subject: WIP disk failure: add persistent disk states .. WIP disk

[kudu-CR] disk failure: tests for disk failure recovery

2017-07-07 Thread Andrew Wong (Code Review)
Hello Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/7243 to look at the new patch set (#5). Change subject: disk failure: tests for disk failure recovery .. disk failure:

[kudu-CR] WIP disk failure: coordinate disk failure handling

2017-07-07 Thread Andrew Wong (Code Review)
Hello Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/7030 to look at the new patch set (#11). Change subject: WIP disk failure: coordinate disk failure handling .. WIP disk

[kudu-CR] mini cluster: support multiple data dirs

2017-07-07 Thread Andrew Wong (Code Review)
Hello Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/7211 to look at the new patch set (#12). Change subject: mini_cluster: support multiple data dirs .. mini_cluster:

[kudu-CR] disk failure: coordinate error handling

2017-07-07 Thread Andrew Wong (Code Review)
Hello Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/7029 to look at the new patch set (#17). Change subject: disk failure: coordinate error handling .. disk failure:

[kudu-CR] [client samples-test] adapt for 'absent lsof' case

2017-07-07 Thread Alexey Serbin (Code Review)
Alexey Serbin has posted comments on this change. Change subject: [client_samples-test] adapt for 'absent lsof' case .. Patch Set 1: Verified+1 Unrelated breakage in raft_consensus-itest -- To view, visit

[kudu-CR] [java] KUDU-2013 re-acquire authn token if expired

2017-07-07 Thread Alexey Serbin (Code Review)
Alexey Serbin has posted comments on this change. Change subject: [java] KUDU-2013 re-acquire authn token if expired .. Patch Set 9: Verified+1 Unrelated flake in MultiThreadedTabletTest/1.DeleteAndReinsert -- To view, visit

[kudu-CR] [WIP]log block manager:update container in-memory accounting after data is synced

2017-07-07 Thread Andrew Wong (Code Review)
Andrew Wong has posted comments on this change. Change subject: [WIP]log block manager:update container in-memory accounting after data is synced .. Patch Set 5: (9 comments)

[kudu-CR] [client samples-test] wait for tserver registration

2017-07-07 Thread Alexey Serbin (Code Review)
Alexey Serbin has submitted this change and it was merged. Change subject: [client_samples-test] wait for tserver registration .. [client_samples-test] wait for tserver registration Wait for the tserver to register with the

[kudu-CR] [WIP]log block manager:update container in-memory accounting after data is synced

2017-07-07 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 (#5). Change subject: [WIP]log block manager:update container in-memory accounting after data is synced

[kudu-CR] [client samples-test] wait for tserver registration

2017-07-07 Thread Dan Burkert (Code Review)
Dan Burkert has posted comments on this change. Change subject: [client_samples-test] wait for tserver registration .. Patch Set 4: Code-Review+2 -- To view, visit http://gerrit.cloudera.org:8080/7378 To unsubscribe, visit

[kudu-CR] [client][tests] fixed misprints

2017-07-07 Thread Alexey Serbin (Code Review)
Alexey Serbin has submitted this change and it was merged. Change subject: [client][tests] fixed misprints .. [client][tests] fixed misprints Fixed misprint in warning message and test comments: READ_AT_TIMESTAMP -->

[kudu-CR] [client][tests] fixed misprints

2017-07-07 Thread Alexey Serbin (Code Review)
Alexey Serbin has posted comments on this change. Change subject: [client][tests] fixed misprints .. Patch Set 2: Verified+1 Hit the issue addressed by http://gerrit.cloudera.org:8080/7378 -- To view, visit

[kudu-CR] [client samples-test] wait for tserver registration

2017-07-07 Thread Alexey Serbin (Code Review)
Hello Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/7378 to look at the new patch set (#4). Change subject: [client_samples-test] wait for tserver registration ..

[kudu-CR] [WIP]log block manager:update container in-memory accounting after data is synced

2017-07-07 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 (#4). Change subject: [WIP]log block manager:update container in-memory accounting after data is synced

[kudu-CR] [WIP]log block manager:update container in-memory accounting after data is synced

2017-07-07 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 (#3). Change subject: [WIP]log block manager:update container in-memory accounting after data is synced

[kudu-CR] [client samples-test] wait for tserver registration

2017-07-07 Thread Alexey Serbin (Code Review)
Alexey Serbin has posted comments on this change. Change subject: [client_samples-test] wait for tserver registration .. Patch Set 3: (1 comment)

[kudu-CR] [client][tests] fixed misprints

2017-07-07 Thread Dan Burkert (Code Review)
Dan Burkert has posted comments on this change. Change subject: [client][tests] fixed misprints .. Patch Set 2: Code-Review+2 -- To view, visit http://gerrit.cloudera.org:8080/7377 To unsubscribe, visit

[kudu-CR] [client samples-test] wait for tserver registration

2017-07-07 Thread Dan Burkert (Code Review)
Dan Burkert has posted comments on this change. Change subject: [client_samples-test] wait for tserver registration .. Patch Set 3: (1 comment) http://gerrit.cloudera.org:8080/#/c/7378/3/src/kudu/client/client_samples-test.sh

[kudu-CR] [client][tests] fixed misprints

2017-07-07 Thread Alexey Serbin (Code Review)
Alexey Serbin has uploaded a new patch set (#2). Change subject: [client][tests] fixed misprints .. [client][tests] fixed misprints Fixed misprint in warning message and test comments: READ_AT_TIMESTAMP --> READ_AT_SNAPSHOT

[kudu-CR] [client samples-test] wait for tserver registration

2017-07-07 Thread Alexey Serbin (Code Review)
Hello Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/7378 to look at the new patch set (#3). Change subject: [client_samples-test] wait for tserver registration ..

[kudu-CR] KUDU-1865 (part 1): reduce some cross-thread allocations

2017-07-07 Thread Michael Ho (Code Review)
Michael Ho has posted comments on this change. Change subject: KUDU-1865 (part 1): reduce some cross-thread allocations .. Patch Set 4: (1 comment) http://gerrit.cloudera.org:8080/#/c/5905/4/src/kudu/rpc/connection.cc File

[kudu-CR] [client samples-test] wait for tserver registration

2017-07-07 Thread Alexey Serbin (Code Review)
Hello Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/7378 to look at the new patch set (#2). Change subject: [client_samples-test] wait for tserver registration ..

[kudu-CR] [client samples-test] wait for tserver registration

2017-07-07 Thread Alexey Serbin (Code Review)
Alexey Serbin has posted comments on this change. Change subject: [client_samples-test] wait for tserver registration .. Patch Set 1: (1 comment)

[kudu-CR] [test] extra time for tserver registration

2017-07-07 Thread Dan Burkert (Code Review)
Dan Burkert has posted comments on this change. Change subject: [test] extra time for tserver registration .. Patch Set 1: (1 comment) http://gerrit.cloudera.org:8080/#/c/7378/1/src/kudu/client/client_samples-test.sh File

[kudu-CR] [test] extra time for tserver registration

2017-07-07 Thread Alexey Serbin (Code Review)
Alexey Serbin has uploaded a new change for review. http://gerrit.cloudera.org:8080/7378 Change subject: [test] extra time for tserver registration .. [test] extra time for tserver registration Added extra delay prior to

[kudu-CR] [client][tests] fixed misprints

2017-07-07 Thread Alexey Serbin (Code Review)
Alexey Serbin has uploaded a new change for review. http://gerrit.cloudera.org:8080/7377 Change subject: [client][tests] fixed misprints .. [client][tests] fixed misprints Fixed misprint in warning message and test comments:

[kudu-CR] WIP disk failure: coordinate disk failure handling

2017-07-07 Thread Andrew Wong (Code Review)
Andrew Wong has posted comments on this change. Change subject: WIP disk failure: coordinate disk failure handling .. Patch Set 9: (23 comments) http://gerrit.cloudera.org:8080/#/c/7030/9/src/kudu/fs/block_manager.h File

[kudu-CR] WIP disk failure: coordinate disk failure handling

2017-07-07 Thread Andrew Wong (Code Review)
Hello Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/7030 to look at the new patch set (#10). Change subject: WIP disk failure: coordinate disk failure handling .. WIP disk

[kudu-CR] [net util] fix reporting getaddrinfo() error

2017-07-07 Thread Alexey Serbin (Code Review)
Alexey Serbin has submitted this change and it was merged. Change subject: [net_util] fix reporting getaddrinfo() error .. [net_util] fix reporting getaddrinfo() error An example of error output: on Linux, prior to this fix:

[kudu-CR] [net util] fix reporting getaddrinfo() error

2017-07-07 Thread Alexey Serbin (Code Review)
Alexey Serbin has posted comments on this change. Change subject: [net_util] fix reporting getaddrinfo() error .. Patch Set 4: Verified+1 Unrelated flake in client_samples-test -- I will take a look at that. As it seen from

[kudu-CR] [WIP]log block manager:update container in-memory accounting after data is synced

2017-07-07 Thread Hao Hao (Code Review)
Hao Hao has uploaded a new patch set (#2). Change subject: [WIP]log block manager:update container in-memory accounting after data is synced .. [WIP]log block manager:update container in-memory accounting after data is synced

[kudu-CR] log block manager:update container in-memory accounting after data is synced

2017-07-07 Thread Hao Hao (Code Review)
Hao Hao has uploaded a new change for review. http://gerrit.cloudera.org:8080/7374 Change subject: log block manager:update container in-memory accounting after data is synced .. log block manager:update container in-memory