[kudu-CR] KUDU-1358 (part 1): master should accept heartbeat even if follower

2016-07-10 Thread Adar Dembo (Code Review)
Adar Dembo has posted comments on this change. Change subject: KUDU-1358 (part 1): master should accept heartbeat even if follower .. Patch Set 1: Filed KUDU-1521 for the Java test failure. -- To view, visit

[kudu-CR] c++ client: various fixes to DDL operations

2016-07-10 Thread David Ribeiro Alves (Code Review)
David Ribeiro Alves has posted comments on this change. Change subject: c++ client: various fixes to DDL operations .. Patch Set 1: (2 comments) http://gerrit.cloudera.org:8080/#/c/3608/1/src/kudu/client/client-internal.cc

[kudu-CR] master-test: rewrite to use std::thread instead of kudu::Thread

2016-07-10 Thread Adar Dembo (Code Review)
Adar Dembo has submitted this change and it was merged. Change subject: master-test: rewrite to use std::thread instead of kudu::Thread .. master-test: rewrite to use std::thread instead of kudu::Thread It's more terse than

[kudu-CR] master: add read-write lock to serialize operations around elections

2016-07-10 Thread Adar Dembo (Code Review)
Adar Dembo has posted comments on this change. Change subject: master: add read-write lock to serialize operations around elections .. Patch Set 4: (2 comments)

[kudu-CR] rw mutex: add configurable priority

2016-07-10 Thread Kudu Jenkins (Code Review)
Kudu Jenkins has posted comments on this change. Change subject: rw_mutex: add configurable priority .. Patch Set 2: -Verified Build Started http://104.196.14.100/job/kudu-gerrit/2300/ -- To view, visit

[kudu-CR] master: fix initialization race with consensus RPCs

2016-07-10 Thread Kudu Jenkins (Code Review)
Kudu Jenkins has posted comments on this change. Change subject: master: fix initialization race with consensus RPCs .. Patch Set 2: Build Started http://104.196.14.100/job/kudu-gerrit/2295/ -- To view, visit

[kudu-CR] rw mutex: add try lock methods

2016-07-10 Thread Kudu Jenkins (Code Review)
Kudu Jenkins has posted comments on this change. Change subject: rw_mutex: add try lock methods .. Patch Set 2: Build Started http://104.196.14.100/job/kudu-gerrit/2297/ -- To view, visit http://gerrit.cloudera.org:8080/3604

[kudu-CR] master: fix corruption when AlterTable() races with CreateTable()

2016-07-10 Thread Kudu Jenkins (Code Review)
Kudu Jenkins has posted comments on this change. Change subject: master: fix corruption when AlterTable() races with CreateTable() .. Patch Set 2: Build Started http://104.196.14.100/job/kudu-gerrit/2298/ -- To view, visit

[kudu-CR] master: add read-write lock to serialize operations around elections

2016-07-10 Thread Adar Dembo (Code Review)
Hello Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/3550 to look at the new patch set (#5). Change subject: master: add read-write lock to serialize operations around elections

[kudu-CR] rw mutex: add configurable priority

2016-07-10 Thread Adar Dembo (Code Review)
Hello Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/3603 to look at the new patch set (#2). Change subject: rw_mutex: add configurable priority .. rw_mutex: add

[kudu-CR] c++ client: various fixes to DDL operations

2016-07-10 Thread Kudu Jenkins (Code Review)
Kudu Jenkins has posted comments on this change. Change subject: c++ client: various fixes to DDL operations .. Patch Set 2: Build Started http://104.196.14.100/job/kudu-gerrit/2292/ -- To view, visit

[kudu-CR] rw mutex: add try lock methods

2016-07-10 Thread Adar Dembo (Code Review)
Hello Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/3604 to look at the new patch set (#2). Change subject: rw_mutex: add try lock methods .. rw_mutex: add try lock

[kudu-CR] master: add read-write lock to serialize operations around elections

2016-07-10 Thread Kudu Jenkins (Code Review)
Kudu Jenkins has posted comments on this change. Change subject: master: add read-write lock to serialize operations around elections .. Patch Set 5: Build Started http://104.196.14.100/job/kudu-gerrit/2296/ -- To view,

[kudu-CR] master: reduce timeout for master to tserver rpcs

2016-07-10 Thread Kudu Jenkins (Code Review)
Kudu Jenkins has posted comments on this change. Change subject: master: reduce timeout for master to tserver rpcs .. Patch Set 2: Build Started http://104.196.14.100/job/kudu-gerrit/2299/ -- To view, visit

[kudu-CR] KUDU-1358 (part 1): master should accept heartbeat even if follower

2016-07-10 Thread Kudu Jenkins (Code Review)
Kudu Jenkins has posted comments on this change. Change subject: KUDU-1358 (part 1): master should accept heartbeat even if follower .. Patch Set 2: Build Started http://104.196.14.100/job/kudu-gerrit/2291/ -- To view,

[kudu-CR] KUDU-1358 (part 2): heartbeat to every master

2016-07-10 Thread Kudu Jenkins (Code Review)
Kudu Jenkins has posted comments on this change. Change subject: KUDU-1358 (part 2): heartbeat to every master .. Patch Set 2: Build Started http://104.196.14.100/job/kudu-gerrit/2294/ -- To view, visit

[kudu-CR] rw mutex: add configurable priority

2016-07-10 Thread Adar Dembo (Code Review)
Adar Dembo has posted comments on this change. Change subject: rw_mutex: add configurable priority .. Patch Set 1: (2 comments) http://gerrit.cloudera.org:8080/#/c/3603/1/src/kudu/util/rw_mutex.h File

[kudu-CR] master: fix corruption when AlterTable() races with CreateTable()

2016-07-10 Thread Adar Dembo (Code Review)
Adar Dembo has posted comments on this change. Change subject: master: fix corruption when AlterTable() races with CreateTable() .. Patch Set 1: (2 comments) The test failure was in the initial write to ITScannerMultiTablet.