[kudu-CR](gh-pages) Replace bootstrap with CDN version

2016-07-14 Thread Jean-Daniel Cryans (Code Review)
Jean-Daniel Cryans has posted comments on this change. Change subject: Replace bootstrap with CDN version .. Patch Set 1: Code-Review+2 -- To view, visit http://gerrit.cloudera.org:8080/3649 To unsubscribe, visit

[kudu-CR] KUDU-1311 [master] support adding and dropping range partitions

2016-07-14 Thread Dan Burkert (Code Review)
Hello Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/3648 to look at the new patch set (#3). Change subject: KUDU-1311 [master] support adding and dropping range partitions

[kudu-CR] Fix encoding-test on OS X

2016-07-14 Thread Dan Burkert (Code Review)
Dan Burkert has submitted this change and it was merged. Change subject: Fix encoding-test on OS X .. Fix encoding-test on OS X A recent change caused encoding-test to fail on OS X. This reverts the part of the change that

[kudu-CR] KUDU-1487 Add instructions for cutting a release

2016-07-14 Thread Kudu Jenkins (Code Review)
Kudu Jenkins has posted comments on this change. Change subject: KUDU-1487 Add instructions for cutting a release .. Patch Set 6: Build Started http://104.196.14.100/job/kudu-gerrit/2432/ -- To view, visit

[kudu-CR] KUDU-1311 [master] support adding and dropping range partitions

2016-07-14 Thread Jean-Daniel Cryans (Code Review)
Jean-Daniel Cryans has posted comments on this change. Change subject: KUDU-1311 [master] support adding and dropping range partitions .. Patch Set 1: (2 comments)

[kudu-CR] KUDU-1311 [master] support adding and dropping range partitions

2016-07-14 Thread Kudu Jenkins (Code Review)
Kudu Jenkins has posted comments on this change. Change subject: KUDU-1311 [master] support adding and dropping range partitions .. Patch Set 2: Build Started http://104.196.14.100/job/kudu-gerrit/2435/ -- To view, visit

[kudu-CR] KUDU-1311 [master] support adding and dropping range partitions

2016-07-14 Thread Dan Burkert (Code Review)
Dan Burkert has posted comments on this change. Change subject: KUDU-1311 [master] support adding and dropping range partitions .. Patch Set 1: (2 comments) http://gerrit.cloudera.org:8080/#/c/3648/1/src/kudu/client/client.cc

[kudu-CR] [c++-client]: cache non-covering ranges in meta cache

2016-07-14 Thread Jean-Daniel Cryans (Code Review)
Jean-Daniel Cryans has posted comments on this change. Change subject: [c++-client]: cache non-covering ranges in meta cache .. Patch Set 4: (2 comments)

[kudu-CR] docs: Fix broken link to Introduction in navbar

2016-07-14 Thread Mike Percy (Code Review)
Mike Percy has submitted this change and it was merged. Change subject: docs: Fix broken link to Introduction in navbar .. docs: Fix broken link to Introduction in navbar We moved introduction.adoc to index.adoc, but the

[kudu-CR] KUDU-1487 Add instructions for cutting a release

2016-07-14 Thread Jean-Daniel Cryans (Code Review)
Hello Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/3614 to look at the new patch set (#6). Change subject: KUDU-1487 Add instructions for cutting a release .. KUDU-1487

[kudu-CR] Change version from 1.0.0-SNAPSHOT to 0.10.0-SNAPSHOT

2016-07-14 Thread Jean-Daniel Cryans (Code Review)
Jean-Daniel Cryans has uploaded a new change for review. http://gerrit.cloudera.org:8080/3651 Change subject: Change version from 1.0.0-SNAPSHOT to 0.10.0-SNAPSHOT .. Change version from 1.0.0-SNAPSHOT to 0.10.0-SNAPSHOT

[kudu-CR] Change version from 1.0.0-SNAPSHOT to 0.10.0-SNAPSHOT

2016-07-14 Thread Kudu Jenkins (Code Review)
Kudu Jenkins has posted comments on this change. Change subject: Change version from 1.0.0-SNAPSHOT to 0.10.0-SNAPSHOT .. Patch Set 1: Build Started http://104.196.14.100/job/kudu-gerrit/2433/ -- To view, visit

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

2016-07-14 Thread Dan Burkert (Code Review)
Dan Burkert has posted comments on this change. Change subject: KUDU-1358 (part 1): master should accept heartbeat even if follower .. Patch Set 6: Code-Review+1 -- To view, visit http://gerrit.cloudera.org:8080/3609 To

[kudu-CR] [c++-client]: cache non-covering ranges in meta cache

2016-07-14 Thread Dan Burkert (Code Review)
Dan Burkert has posted comments on this change. Change subject: [c++-client]: cache non-covering ranges in meta cache .. Patch Set 5: (2 comments) http://gerrit.cloudera.org:8080/#/c/3581/4/src/kudu/client/meta_cache.cc File

[kudu-CR] KUDU-1311 [master] support adding and dropping range partitions

2016-07-14 Thread Dan Burkert (Code Review)
Hello Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/3648 to look at the new patch set (#2). Change subject: KUDU-1311 [master] support adding and dropping range partitions

[kudu-CR] Change version from 1.0.0-SNAPSHOT to 0.10.0-SNAPSHOT

2016-07-14 Thread Mike Percy (Code Review)
Mike Percy has posted comments on this change. Change subject: Change version from 1.0.0-SNAPSHOT to 0.10.0-SNAPSHOT .. Patch Set 1: Code-Review+2 -- To view, visit http://gerrit.cloudera.org:8080/3651 To unsubscribe, visit

[kudu-CR] rw mutex: add configurable priority

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

[kudu-CR] Fix flaky disk reservation-itest

2016-07-14 Thread Kudu Jenkins (Code Review)
Kudu Jenkins has posted comments on this change. Change subject: Fix flaky disk_reservation-itest .. Patch Set 1: Build Started http://104.196.14.100/job/kudu-gerrit/2439/ -- To view, visit

[kudu-CR] Fix flaky disk reservation-itest

2016-07-14 Thread Mike Percy (Code Review)
Hello Todd Lipcon, I'd like you to do a code review. Please visit http://gerrit.cloudera.org:8080/3652 to review the following change. Change subject: Fix flaky disk_reservation-itest .. Fix flaky disk_reservation-itest

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

2016-07-14 Thread Dan Burkert (Code Review)
Dan Burkert has posted comments on this change. Change subject: master: fix initialization race with consensus RPCs .. Patch Set 5: Code-Review+2 -- To view, visit http://gerrit.cloudera.org:8080/3605 To unsubscribe, visit

[kudu-CR] Add integration tests for replay cache with writes

2016-07-14 Thread David Ribeiro Alves (Code Review)
David Ribeiro Alves has posted comments on this change. Change subject: Add integration tests for replay cache with writes .. Patch Set 29: (8 comments) i moved these to a new itest, also moved a couple of methods out of raft

[kudu-CR] Add integration tests for replay cache with writes

2016-07-14 Thread David Ribeiro Alves (Code Review)
David Ribeiro Alves has uploaded a new patch set (#30). Change subject: Add integration tests for replay cache with writes .. Add integration tests for replay cache with writes This adds a couple of new integration tests for

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

2016-07-14 Thread Dan Burkert (Code Review)
Dan Burkert has posted comments on this change. Change subject: master: add read-write lock to serialize operations around elections .. Patch Set 8: Code-Review+2 -- To view, visit http://gerrit.cloudera.org:8080/3550 To

[kudu-CR] master: do not delete unknown tablets

2016-07-14 Thread Kudu Jenkins (Code Review)
Kudu Jenkins has posted comments on this change. Change subject: master: do not delete unknown tablets .. Patch Set 3: Build Started http://104.196.14.100/job/kudu-gerrit/2443/ -- To view, visit

[kudu-CR] master: add assert checks for leader lock

2016-07-14 Thread Kudu Jenkins (Code Review)
Kudu Jenkins has posted comments on this change. Change subject: master: add assert checks for leader_lock .. Patch Set 3: Build Started http://104.196.14.100/job/kudu-gerrit/2447/ -- To view, visit

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

2016-07-14 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 7: Build Started http://104.196.14.100/job/kudu-gerrit/2445/ -- To view,

[kudu-CR] rw mutex: prevent recursive use

2016-07-14 Thread Adar Dembo (Code Review)
Hello Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/3641 to look at the new patch set (#3). Change subject: rw_mutex: prevent recursive use .. rw_mutex: prevent recursive

[kudu-CR] rw mutex: prevent recursive use

2016-07-14 Thread Kudu Jenkins (Code Review)
Kudu Jenkins has posted comments on this change. Change subject: rw_mutex: prevent recursive use .. Patch Set 2: Build Started http://104.196.14.100/job/kudu-gerrit/2449/ -- To view, visit

[kudu-CR] rw mutex: add configurable priority

2016-07-14 Thread Dan Burkert (Code Review)
Dan Burkert has posted comments on this change. Change subject: rw_mutex: add configurable priority .. Patch Set 4: Code-Review+2 -- To view, visit http://gerrit.cloudera.org:8080/3603 To unsubscribe, visit

[kudu-CR] master: add assert checks for leader lock

2016-07-14 Thread Dan Burkert (Code Review)
Dan Burkert has posted comments on this change. Change subject: master: add assert checks for leader_lock .. Patch Set 2: Code-Review+2 -- To view, visit http://gerrit.cloudera.org:8080/3642 To unsubscribe, visit

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

2016-07-14 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 8: Build Started http://104.196.14.100/job/kudu-gerrit/2462/ -- To view, visit

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

2016-07-14 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 10: Build Started http://104.196.14.100/job/kudu-gerrit/2464/ -- To view,

[kudu-CR] master: add assert checks for leader lock

2016-07-14 Thread Kudu Jenkins (Code Review)
Kudu Jenkins has posted comments on this change. Change subject: master: add assert checks for leader_lock .. Patch Set 4: Build Started http://104.196.14.100/job/kudu-gerrit/2459/ -- To view, visit

[kudu-CR] KUDU-1358 (part 3): new multi-master stress test

2016-07-14 Thread Kudu Jenkins (Code Review)
Kudu Jenkins has posted comments on this change. Change subject: KUDU-1358 (part 3): new multi-master stress test .. Patch Set 8: Build Started http://104.196.14.100/job/kudu-gerrit/2460/ -- To view, visit

[kudu-CR] rw mutex: prevent recursive use

2016-07-14 Thread Kudu Jenkins (Code Review)
Kudu Jenkins has posted comments on this change. Change subject: rw_mutex: prevent recursive use .. Patch Set 3: Build Started http://104.196.14.100/job/kudu-gerrit/2461/ -- To view, visit

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

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

[kudu-CR] KUDU-1374: send full tablet report when new leader master is detected

2016-07-14 Thread Kudu Jenkins (Code Review)
Kudu Jenkins has posted comments on this change. Change subject: KUDU-1374: send full tablet report when new leader master is detected .. Patch Set 4: Build Started http://104.196.14.100/job/kudu-gerrit/2456/ -- To view,

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

2016-07-14 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 7: Build Started http://104.196.14.100/job/kudu-gerrit/2465/ -- To view, visit

[kudu-CR] rw mutex: prevent recursive use

2016-07-14 Thread Dan Burkert (Code Review)
Dan Burkert has posted comments on this change. Change subject: rw_mutex: prevent recursive use .. Patch Set 1: (1 comment) http://gerrit.cloudera.org:8080/#/c/3641/1/src/kudu/util/rw_mutex.h File src/kudu/util/rw_mutex.h:

[kudu-CR] rw mutex: add configurable priority

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

[kudu-CR] KUDU-1311 [master] support adding and dropping range partitions

2016-07-14 Thread Kudu Jenkins (Code Review)
Kudu Jenkins has posted comments on this change. Change subject: KUDU-1311 [master] support adding and dropping range partitions .. Patch Set 4: Build Started http://104.196.14.100/job/kudu-gerrit/2438/ -- To view, visit

[kudu-CR] master: add assert checks for leader lock

2016-07-14 Thread Dan Burkert (Code Review)
Dan Burkert has posted comments on this change. Change subject: master: add assert checks for leader_lock .. Patch Set 4: Code-Review+2 -- To view, visit http://gerrit.cloudera.org:8080/3642 To unsubscribe, visit

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

2016-07-14 Thread Dan Burkert (Code Review)
Dan Burkert has posted comments on this change. Change subject: master: fix corruption when AlterTable() races with CreateTable() .. Patch Set 7: Code-Review+2 -- To view, visit http://gerrit.cloudera.org:8080/3607 To

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

2016-07-14 Thread Dan Burkert (Code Review)
Dan Burkert has posted comments on this change. Change subject: c++ client: various fixes to DDL operations .. Patch Set 8: Code-Review+2 -- To view, visit http://gerrit.cloudera.org:8080/3608 To unsubscribe, visit

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

2016-07-14 Thread Dan Burkert (Code Review)
Dan Burkert has posted comments on this change. Change subject: master: fix initialization race with consensus RPCs .. Patch Set 7: Code-Review+2 -- To view, visit http://gerrit.cloudera.org:8080/3605 To unsubscribe, visit

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

2016-07-14 Thread Dan Burkert (Code Review)
Dan Burkert has posted comments on this change. Change subject: master: add read-write lock to serialize operations around elections .. Patch Set 10: Code-Review+2 -- To view, visit http://gerrit.cloudera.org:8080/3550 To

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

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

[kudu-CR] WIP [java-client] Re-enable multi-master tests

2016-07-14 Thread Jean-Daniel Cryans (Code Review)
Jean-Daniel Cryans has uploaded a new change for review. http://gerrit.cloudera.org:8080/3654 Change subject: WIP [java-client] Re-enable multi-master tests .. WIP [java-client] Re-enable multi-master tests This patch makes

[kudu-CR] WIP [java-client] Re-enable multi-master tests

2016-07-14 Thread Kudu Jenkins (Code Review)
Kudu Jenkins has posted comments on this change. Change subject: WIP [java-client] Re-enable multi-master tests .. Patch Set 1: Build Started http://104.196.14.100/job/kudu-gerrit/2468/ -- To view, visit

[kudu-CR] WIP: Add garbage collection to ResultTracker

2016-07-14 Thread David Ribeiro Alves (Code Review)
Hello Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/3628 to look at the new patch set (#5). Change subject: WIP: Add garbage collection to ResultTracker .. WIP: Add

[kudu-CR] [c++-client]: cache non-covering ranges in meta cache

2016-07-14 Thread Adar Dembo (Code Review)
Adar Dembo has posted comments on this change. Change subject: [c++-client]: cache non-covering ranges in meta cache .. Patch Set 5: (17 comments) http://gerrit.cloudera.org:8080/#/c/3581/5/src/kudu/client/client-test.cc File

[kudu-CR] KUDU-1358 (part 3): new multi-master stress test

2016-07-14 Thread Adar Dembo (Code Review)
Adar Dembo has posted comments on this change. Change subject: KUDU-1358 (part 3): new multi-master stress test .. Patch Set 8: Verified+1 Another instance of KUDU-1527. -- To view, visit http://gerrit.cloudera.org:8080/3611

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

2016-07-14 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 7: Verified+1 Isolate crashed. -- To view, visit

[kudu-CR] WIP [java-client] Re-enable multi-master tests

2016-07-14 Thread Jean-Daniel Cryans (Code Review)
Jean-Daniel Cryans has posted comments on this change. Change subject: WIP [java-client] Re-enable multi-master tests .. Patch Set 1: (2 comments)

[kudu-CR] rw mutex: prevent recursive use

2016-07-14 Thread Kudu Jenkins (Code Review)
Kudu Jenkins has posted comments on this change. Change subject: rw_mutex: prevent recursive use .. Patch Set 4: Build Started http://104.196.14.100/job/kudu-gerrit/2470/ -- To view, visit

[kudu-CR] rw mutex: prevent recursive use

2016-07-14 Thread Adar Dembo (Code Review)
Hello Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/3641 to look at the new patch set (#4). Change subject: rw_mutex: prevent recursive use .. rw_mutex: prevent recursive

[kudu-CR] KUDU-1530: Update docs about OS X build dependency on Xcode package

2016-07-14 Thread Dan Burkert (Code Review)
Dan Burkert has posted comments on this change. Change subject: KUDU-1530: Update docs about OS X build dependency on Xcode package .. Patch Set 1: (1 comment)

[kudu-CR] WIP: Add garbage collection to ResultTracker

2016-07-14 Thread Jean-Daniel Cryans (Code Review)
Jean-Daniel Cryans has posted comments on this change. Change subject: WIP: Add garbage collection to ResultTracker .. Patch Set 5: (3 comments) http://gerrit.cloudera.org:8080/#/c/3628/5/src/kudu/rpc/result_tracker.cc File

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

2016-07-14 Thread Adar Dembo (Code Review)
Adar Dembo has posted comments on this change. Change subject: c++ client: various fixes to DDL operations .. Patch Set 8: Verified+1 A whole slew of chrpath() failures in dist-test. -- To view, visit

[kudu-CR] master: add assert checks for leader lock

2016-07-14 Thread Adar Dembo (Code Review)
Adar Dembo has posted comments on this change. Change subject: master: add assert checks for leader_lock .. Patch Set 4: Verified+1 Another chrpath() failure in dist-test. -- To view, visit

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

2016-07-14 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 8: Code-Review+2 -- To view, visit http://gerrit.cloudera.org:8080/3608 To unsubscribe, visit

[kudu-CR] rw mutex: prevent recursive use

2016-07-14 Thread Dan Burkert (Code Review)
Dan Burkert has posted comments on this change. Change subject: rw_mutex: prevent recursive use .. Patch Set 1: (1 comment) http://gerrit.cloudera.org:8080/#/c/3641/1/src/kudu/util/rw_mutex.h File src/kudu/util/rw_mutex.h:

[kudu-CR] WIP: Add garbage collection to ResultTracker

2016-07-14 Thread David Ribeiro Alves (Code Review)
Hello Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/3628 to look at the new patch set (#4). Change subject: WIP: Add garbage collection to ResultTracker .. WIP: Add

[kudu-CR] KUDU-1530: Update docs about OS X build dependency on Xcode package

2016-07-14 Thread Dinesh Bhat (Code Review)
Dinesh Bhat has uploaded a new change for review. http://gerrit.cloudera.org:8080/3653 Change subject: KUDU-1530: Update docs about OS X build dependency on Xcode package .. KUDU-1530: Update docs about OS X build dependency

[kudu-CR] WIP: Add garbage collection to ResultTracker

2016-07-14 Thread Kudu Jenkins (Code Review)
Kudu Jenkins has posted comments on this change. Change subject: WIP: Add garbage collection to ResultTracker .. Patch Set 4: Build Started http://104.196.14.100/job/kudu-gerrit/2467/ -- To view, visit

[kudu-CR] WIP [java-client] Re-enable multi-master tests

2016-07-14 Thread Adar Dembo (Code Review)
Adar Dembo has posted comments on this change. Change subject: WIP [java-client] Re-enable multi-master tests .. Patch Set 1: (2 comments)

[kudu-CR] rw mutex: prevent recursive use

2016-07-14 Thread Adar Dembo (Code Review)
Adar Dembo has posted comments on this change. Change subject: rw_mutex: prevent recursive use .. Patch Set 1: (1 comment) http://gerrit.cloudera.org:8080/#/c/3641/1/src/kudu/util/rw_mutex.h File src/kudu/util/rw_mutex.h:

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

2016-07-14 Thread Adar Dembo (Code Review)
Hello Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/3610 to look at the new patch set (#9). Change subject: KUDU-1358 (part 2): heartbeat to every master .. KUDU-1358

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

2016-07-14 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 9: Build Started http://104.196.14.100/job/kudu-gerrit/2477/ -- To view, visit

[kudu-CR] Add a way to dump ResultTracker state

2016-07-14 Thread Jean-Daniel Cryans (Code Review)
Jean-Daniel Cryans has posted comments on this change. Change subject: Add a way to dump ResultTracker state .. Patch Set 11: Code-Review+2 -- To view, visit http://gerrit.cloudera.org:8080/3569 To unsubscribe, visit

[kudu-CR] Disable exactly once semantics by default and add a flag to enable it for tests

2016-07-14 Thread David Ribeiro Alves (Code Review)
Hello Jean-Daniel Cryans, Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/3506 to look at the new patch set (#16). Change subject: Disable exactly once semantics by default and add a flag to enable it for tests

[kudu-CR] Disable exactly once semantics by default and add a flag to enable it for tests

2016-07-14 Thread David Ribeiro Alves (Code Review)
David Ribeiro Alves has posted comments on this change. Change subject: Disable exactly once semantics by default and add a flag to enable it for tests .. Patch Set 15: (1 comment) adressed the nit, keeping the +2

[kudu-CR] Move the maintenance manager to util

2016-07-14 Thread Kudu Jenkins (Code Review)
Kudu Jenkins has posted comments on this change. Change subject: Move the maintenance manager to util .. Patch Set 1: Build Started http://104.196.14.100/job/kudu-gerrit/2480/ -- To view, visit

[kudu-CR] doxygen for C++ client API

2016-07-14 Thread Kudu Jenkins (Code Review)
Kudu Jenkins has posted comments on this change. Change subject: doxygen for C++ client API .. Patch Set 7: Build Started http://104.196.14.100/job/kudu-gerrit/2503/ -- To view, visit http://gerrit.cloudera.org:8080/3619 To

[kudu-CR] Add integration tests for replay cache with writes

2016-07-14 Thread David Ribeiro Alves (Code Review)
David Ribeiro Alves has submitted this change and it was merged. Change subject: Add integration tests for replay cache with writes .. Add integration tests for replay cache with writes This adds a couple of new integration

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

2016-07-14 Thread Adar Dembo (Code Review)
Adar Dembo has submitted this change and it was merged. Change subject: master: add read-write lock to serialize operations around elections .. master: add read-write lock to serialize operations around elections This

[kudu-CR] Integrate the ResultTracker into the rpc subsystem

2016-07-14 Thread David Ribeiro Alves (Code Review)
David Ribeiro Alves has submitted this change and it was merged. Change subject: Integrate the ResultTracker into the rpc subsystem .. Integrate the ResultTracker into the rpc subsystem This integrates the ResultTracker into

[kudu-CR] master: do not delete unknown tablets

2016-07-14 Thread Adar Dembo (Code Review)
Adar Dembo has posted comments on this change. Change subject: master: do not delete unknown tablets .. Patch Set 5: Verified+1 More isolate and chrpath() failures. -- To view, visit http://gerrit.cloudera.org:8080/3645 To

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

2016-07-14 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 9: Verified+1 chrpath() failed. -- To view, visit

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

2016-07-14 Thread Adar Dembo (Code Review)
Adar Dembo has posted comments on this change. Change subject: KUDU-1358 (part 2): heartbeat to every master .. Patch Set 9: Verified+1 Isolate failed. -- To view, visit http://gerrit.cloudera.org:8080/3610 To unsubscribe,

[kudu-CR] rw mutex: prevent recursive use

2016-07-14 Thread Adar Dembo (Code Review)
Adar Dembo has posted comments on this change. Change subject: rw_mutex: prevent recursive use .. Patch Set 5: Verified+1 KUDU-1527 and isolate failures. -- To view, visit http://gerrit.cloudera.org:8080/3641 To unsubscribe,

[kudu-CR] KUDU-1358 (part 3): new multi-master stress test

2016-07-14 Thread Adar Dembo (Code Review)
Adar Dembo has posted comments on this change. Change subject: KUDU-1358 (part 3): new multi-master stress test .. Patch Set 9: Verified+1 chrpath() failures. -- To view, visit http://gerrit.cloudera.org:8080/3611 To

[kudu-CR] Integrate the result tracker with writes

2016-07-14 Thread David Ribeiro Alves (Code Review)
David Ribeiro Alves has posted comments on this change. Change subject: Integrate the result tracker with writes .. Patch Set 25: (13 comments)

[kudu-CR] Integrate the result tracker with writes

2016-07-14 Thread David Ribeiro Alves (Code Review)
David Ribeiro Alves has posted comments on this change. Change subject: Integrate the result tracker with writes .. Patch Set 25: (1 comment)

[kudu-CR] Move the maintenance manager to util

2016-07-14 Thread Adar Dembo (Code Review)
Adar Dembo has posted comments on this change. Change subject: Move the maintenance manager to util .. Patch Set 1: (15 comments) What's the motivation for this move? http://gerrit.cloudera.org:8080/#/c/3656/1//COMMIT_MSG

[kudu-CR] Memory tracking for result tracker

2016-07-14 Thread Jean-Daniel Cryans (Code Review)
Jean-Daniel Cryans has posted comments on this change. Change subject: Memory tracking for result tracker .. Patch Set 6: (1 comment) http://gerrit.cloudera.org:8080/#/c/3627/6//COMMIT_MSG Commit Message: PS6, Line 10:

[kudu-CR] [java client] Integrate with the replay cache

2016-07-14 Thread Kudu Jenkins (Code Review)
Kudu Jenkins has posted comments on this change. Change subject: [java client] Integrate with the replay cache .. Patch Set 4: Build Started http://104.196.14.100/job/kudu-gerrit/2481/ -- To view, visit

[kudu-CR] [java client] Integrate with the replay cache

2016-07-14 Thread Jean-Daniel Cryans (Code Review)
Jean-Daniel Cryans has posted comments on this change. Change subject: [java client] Integrate with the replay cache .. Patch Set 4: (1 comment)

[kudu-CR] Move the maintenance manager to util

2016-07-14 Thread David Ribeiro Alves (Code Review)
David Ribeiro Alves has posted comments on this change. Change subject: Move the maintenance manager to util .. Patch Set 1: (16 comments) This change is required as we'll need a server-wide MM for the result tracker. Added

[kudu-CR] Disable exactly once semantics by default and add a flag to enable it for tests

2016-07-14 Thread David Ribeiro Alves (Code Review)
David Ribeiro Alves has submitted this change and it was merged. Change subject: Disable exactly once semantics by default and add a flag to enable it for tests .. Disable exactly once semantics by default and add a flag to

[kudu-CR] Integrate the result tracker with writes

2016-07-14 Thread Kudu Jenkins (Code Review)
Kudu Jenkins has posted comments on this change. Change subject: Integrate the result tracker with writes .. Patch Set 26: Build Started http://104.196.14.100/job/kudu-gerrit/2484/ -- To view, visit

[kudu-CR] Add integration tests for replay cache with writes

2016-07-14 Thread Kudu Jenkins (Code Review)
Kudu Jenkins has posted comments on this change. Change subject: Add integration tests for replay cache with writes .. Patch Set 31: Build Started http://104.196.14.100/job/kudu-gerrit/2486/ -- To view, visit

[kudu-CR] Add integration tests for replay cache with writes

2016-07-14 Thread David Ribeiro Alves (Code Review)
Hello Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/3519 to look at the new patch set (#31). Change subject: Add integration tests for replay cache with writes .. Add

[kudu-CR] Move the maintenance manager to util

2016-07-14 Thread Kudu Jenkins (Code Review)
Kudu Jenkins has posted comments on this change. Change subject: Move the maintenance manager to util .. Patch Set 2: Build Started http://104.196.14.100/job/kudu-gerrit/2482/ -- To view, visit

[kudu-CR] Memory tracking for result tracker

2016-07-14 Thread Kudu Jenkins (Code Review)
Kudu Jenkins has posted comments on this change. Change subject: Memory tracking for result tracker .. Patch Set 7: Build Started http://104.196.14.100/job/kudu-gerrit/2485/ -- To view, visit

[kudu-CR] Add a way to include request ids in log-dump

2016-07-14 Thread Kudu Jenkins (Code Review)
Kudu Jenkins has posted comments on this change. Change subject: Add a way to include request ids in log-dump .. Patch Set 7: Build Started http://104.196.14.100/job/kudu-gerrit/2483/ -- To view, visit

[kudu-CR] Memory tracking for result tracker

2016-07-14 Thread Kudu Jenkins (Code Review)
Kudu Jenkins has posted comments on this change. Change subject: Memory tracking for result tracker .. Patch Set 8: Build Started http://104.196.14.100/job/kudu-gerrit/2487/ -- To view, visit

[kudu-CR] Memory tracking for result tracker

2016-07-14 Thread David Ribeiro Alves (Code Review)
David Ribeiro Alves has posted comments on this change. Change subject: Memory tracking for result tracker .. Patch Set 6: (1 comment) http://gerrit.cloudera.org:8080/#/c/3627/6//COMMIT_MSG Commit Message: PS6, Line 10:

[kudu-CR] Memory tracking for result tracker

2016-07-14 Thread David Ribeiro Alves (Code Review)
Hello Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/3627 to look at the new patch set (#8). Change subject: Memory tracking for result tracker .. Memory tracking for

[kudu-CR] Add integration tests for replay cache with writes

2016-07-14 Thread David Ribeiro Alves (Code Review)
David Ribeiro Alves has posted comments on this change. Change subject: Add integration tests for replay cache with writes .. Patch Set 24: (1 comment)

  1   2   >