[kudu-CR] env posix: add another failure case to Walk

2017-09-28 Thread Alexey Serbin (Code Review)
Alexey Serbin has posted comments on this change. ( http://gerrit.cloudera.org:8080/8171 ) Change subject: env_posix: add another failure case to Walk .. Patch Set 1: Code-Review+2 (1 comment) After some consideration I

[kudu-CR] env posix: add another failure case to Walk

2017-09-28 Thread Alexey Serbin (Code Review)
Alexey Serbin has posted comments on this change. ( http://gerrit.cloudera.org:8080/8171 ) Change subject: env_posix: add another failure case to Walk .. Patch Set 1: (1 comment)

[kudu-CR] env posix: add another failure case to Walk

2017-09-28 Thread Adar Dembo (Code Review)
Adar Dembo has posted comments on this change. ( http://gerrit.cloudera.org:8080/8171 ) Change subject: env_posix: add another failure case to Walk .. Patch Set 1: (1 comment)

[kudu-CR] env posix: add another failure case to Walk

2017-09-28 Thread Alexey Serbin (Code Review)
Alexey Serbin has posted comments on this change. ( http://gerrit.cloudera.org:8080/8171 ) Change subject: env_posix: add another failure case to Walk .. Patch Set 1: (1 comment)

[kudu-CR] env posix: add another failure case to Walk

2017-09-28 Thread Adar Dembo (Code Review)
Hello Alexey Serbin, Todd Lipcon, I'd like you to do a code review. Please visit http://gerrit.cloudera.org:8080/8171 to review the following change. Change subject: env_posix: add another failure case to Walk ..

[kudu-CR] KUDU-2055 [part 2]: Add util to construct sorted disjoint interval list

2017-09-28 Thread Dan Burkert (Code Review)
Dan Burkert has posted comments on this change. ( http://gerrit.cloudera.org:8080/8041 ) Change subject: KUDU-2055 [part 2]: Add util to construct sorted disjoint interval list .. Patch Set 10: Code-Review+2 -- To view,

[kudu-CR] KUDU-2055 [part 2]: Add util to construct sorted disjoint interval list

2017-09-28 Thread Dan Burkert (Code Review)
Dan Burkert has submitted this change and it was merged. ( http://gerrit.cloudera.org:8080/8041 ) Change subject: KUDU-2055 [part 2]: Add util to construct sorted disjoint interval list .. KUDU-2055 [part 2]: Add util to

[kudu-CR] KUDU-2055 [part 4]: Coalesce hole punch for LBM

2017-09-28 Thread Hao Hao (Code Review)
Hao Hao has posted comments on this change. ( http://gerrit.cloudera.org:8080/8162 ) Change subject: KUDU-2055 [part 4]: Coalesce hole punch for LBM .. Patch Set 2: The failed test seems to be a known flaky test filed in

[kudu-CR] WIP: Pull StatusOr and related tests from protobuf

2017-09-28 Thread Dan Burkert (Code Review)
Dan Burkert has posted comments on this change. ( http://gerrit.cloudera.org:8080/8127 ) Change subject: WIP: Pull StatusOr and related tests from protobuf .. Patch Set 1: In Rust, where this pattern is pervasive, it's common

[kudu-CR] WIP: Pull StatusOr and related tests from protobuf

2017-09-28 Thread Adar Dembo (Code Review)
Adar Dembo has posted comments on this change. ( http://gerrit.cloudera.org:8080/8127 ) Change subject: WIP: Pull StatusOr and related tests from protobuf .. Patch Set 1: At first I thought this was supposed to be a

[kudu-CR] [catalog manager] introduce replica selector

2017-09-28 Thread Alexey Serbin (Code Review)
Alexey Serbin has posted comments on this change. ( http://gerrit.cloudera.org:8080/8161 ) Change subject: [catalog manager] introduce replica selector .. Patch Set 2: (1 comment)

[kudu-CR] KUDU-2055 [part 2]: Add util to construct sorted disjoint interval list

2017-09-28 Thread Hao Hao (Code Review)
Hello Tidy Bot, Alexey Serbin, Dan Burkert, David Ribeiro Alves, Kudu Jenkins, Andrew Wong, Adar Dembo, Todd Lipcon, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/8041 to look at the new patch set (#10). Change subject: KUDU-2055 [part 2]: Add util to

[kudu-CR] KUDU-2055 [part 2]: Add util to construct sorted disjoint interval list

2017-09-28 Thread Dan Burkert (Code Review)
Dan Burkert has posted comments on this change. ( http://gerrit.cloudera.org:8080/8041 ) Change subject: KUDU-2055 [part 2]: Add util to construct sorted disjoint interval list .. Patch Set 9: Code-Review+2 -- To view,

[kudu-CR] [catalog manager] introduce replica selector

2017-09-28 Thread Todd Lipcon (Code Review)
Todd Lipcon has posted comments on this change. ( http://gerrit.cloudera.org:8080/8161 ) Change subject: [catalog manager] introduce replica selector .. Patch Set 2: (1 comment)

[kudu-CR] [catalog manager] introduce replica selector

2017-09-28 Thread Alexey Serbin (Code Review)
Alexey Serbin has uploaded a new patch set (#2). ( http://gerrit.cloudera.org:8080/8161 ) Change subject: [catalog manager] introduce replica selector .. [catalog manager] introduce replica selector Introduced replica selector

[kudu-CR] [webui] Allow custom response codes and headers

2017-09-28 Thread Will Berkeley (Code Review)
Hello Tidy Bot, Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/8141 to look at the new patch set (#4). Change subject: [webui] Allow custom response codes and headers ..

[kudu-CR] KUDU-2055 [part 5]: Incorporate BlockDeletionTransaction in block deletions

2017-09-28 Thread Hao Hao (Code Review)
Hao Hao has uploaded this change for review. ( http://gerrit.cloudera.org:8080/8163 Change subject: KUDU-2055 [part 5]: Incorporate BlockDeletionTransaction in block deletions .. KUDU-2055 [part 5]: Incorporate