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

2016-07-13 Thread Jean-Daniel Cryans (Code Review)
Jean-Daniel Cryans has uploaded a new change for review. http://gerrit.cloudera.org:8080/3631 Change subject: WIP [java client] Integrate with the replay cache .. WIP [java client] Integrate with the replay cache Putting

[kudu-CR] KUDU-1516 ksck should check for more raft-related status issues (partial)

2016-07-13 Thread Kudu Jenkins (Code Review)
Kudu Jenkins has posted comments on this change. Change subject: KUDU-1516 ksck should check for more raft-related status issues (partial) .. Patch Set 1: Build Started http://104.196.14.100/job/kudu-gerrit/2386/ -- To

[kudu-CR] KUDU-1516 ksck should check for more raft-related status issues (partial)

2016-07-13 Thread Will Berkeley (Code Review)
Will Berkeley has uploaded a new change for review. http://gerrit.cloudera.org:8080/3632 Change subject: KUDU-1516 ksck should check for more raft-related status issues (partial) .. KUDU-1516 ksck should check for more

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

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

[kudu-CR] Make block manager-test work on OS X again

2016-07-13 Thread Kudu Jenkins (Code Review)
Kudu Jenkins has posted comments on this change. Change subject: Make block_manager-test work on OS X again .. Patch Set 1: Build Started http://104.196.14.100/job/kudu-gerrit/2387/ -- To view, visit

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

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

[kudu-CR](gh-pages) Make site tool.py support Python 2.6 and Centos 6

2016-07-13 Thread Mike Percy (Code Review)
Mike Percy has posted comments on this change. Change subject: Make site_tool.py support Python 2.6 and Centos 6 .. Patch Set 2: Verified+1 -- To view, visit http://gerrit.cloudera.org:8080/3629 To unsubscribe, visit

[kudu-CR] Make block manager-test work on OS X again

2016-07-13 Thread Kudu Jenkins (Code Review)
Kudu Jenkins has posted comments on this change. Change subject: Make block_manager-test work on OS X again .. Patch Set 2: Build Started http://104.196.14.100/job/kudu-gerrit/2388/ -- To view, visit

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

2016-07-13 Thread Jean-Daniel Cryans (Code Review)
Jean-Daniel Cryans has posted comments on this change. Change subject: KUDU-1487 Add instructions for cutting a release .. Patch Set 1: (1 comment) http://gerrit.cloudera.org:8080/#/c/3614/1/RELEASING.adoc File

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

2016-07-13 Thread Misty Stanley-Jones (Code Review)
Misty Stanley-Jones has posted comments on this change. Change subject: KUDU-1487 Add instructions for cutting a release .. Patch Set 1: (4 comments) http://gerrit.cloudera.org:8080/#/c/3614/1/RELEASING.adoc File

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

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

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

2016-07-13 Thread Will Berkeley (Code Review)
Will Berkeley has uploaded a new change for review. http://gerrit.cloudera.org:8080/3640 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

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

2016-07-13 Thread Kudu Jenkins (Code Review)
Kudu Jenkins has posted comments on this change. Change subject: Fix encoding-test on OS X .. Patch Set 1: Build Started http://104.196.14.100/job/kudu-gerrit/2394/ -- To view, visit http://gerrit.cloudera.org:8080/3640 To

[kudu-CR] [java client] Redo how we manage exceptions

2016-07-13 Thread Jean-Daniel Cryans (Code Review)
Hello Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/3055 to look at the new patch set (#6). Change subject: [java client] Redo how we manage exceptions .. [java client]

[kudu-CR] env: add GetFileSizeOnDiskRecursively

2016-07-13 Thread Kudu Jenkins (Code Review)
Kudu Jenkins has posted comments on this change. Change subject: env: add GetFileSizeOnDiskRecursively .. Patch Set 1: Build Started http://104.196.14.100/job/kudu-gerrit/2410/ -- To view, visit

[kudu-CR] [java client] Redo how we manage exceptions

2016-07-13 Thread Jean-Daniel Cryans (Code Review)
Jean-Daniel Cryans has posted comments on this change. Change subject: [java client] Redo how we manage exceptions .. Patch Set 7: (2 comments) > (3 comments) > > Only a few nits left. > > When you're done, could you

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

2016-07-13 Thread Dan Burkert (Code Review)
Dan Burkert has posted comments on this change. Change subject: Fix encoding-test on OS X .. Patch Set 1: (1 comment) http://gerrit.cloudera.org:8080/#/c/3640/1/src/kudu/cfile/encoding-test.cc File

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

2016-07-13 Thread Will Berkeley (Code Review)
Will Berkeley has posted comments on this change. Change subject: Fix encoding-test on OS X .. Patch Set 1: (1 comment) http://gerrit.cloudera.org:8080/#/c/3640/1/src/kudu/cfile/encoding-test.cc File

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

2016-07-13 Thread Will Berkeley (Code Review)
Hello Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/3640 to look at the new patch set (#2). Change subject: Fix encoding-test on OS X .. Fix encoding-test on OS X A

[kudu-CR] [java client] Redo how we manage exceptions

2016-07-13 Thread Jean-Daniel Cryans (Code Review)
Jean-Daniel Cryans has posted comments on this change. Change subject: [java client] Redo how we manage exceptions .. Patch Set 5: (1 comment)

[kudu-CR] [java client] Redo how we manage exceptions

2016-07-13 Thread Jean-Daniel Cryans (Code Review)
Hello Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/3055 to look at the new patch set (#7). Change subject: [java client] Redo how we manage exceptions .. [java client]

[kudu-CR] [java client] Redo how we manage exceptions

2016-07-13 Thread Kudu Jenkins (Code Review)
Kudu Jenkins has posted comments on this change. Change subject: [java client] Redo how we manage exceptions .. Patch Set 7: Build Started http://104.196.14.100/job/kudu-gerrit/2397/ -- To view, visit

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

2016-07-13 Thread Kudu Jenkins (Code Review)
Kudu Jenkins has posted comments on this change. Change subject: Fix encoding-test on OS X .. Patch Set 2: -Verified Build Started http://104.196.14.100/job/kudu-gerrit/2413/ -- To view, visit

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

2016-07-13 Thread Kudu Jenkins (Code Review)
Kudu Jenkins has posted comments on this change. Change subject: Fix encoding-test on OS X .. Patch Set 2: Build Started http://104.196.14.100/job/kudu-gerrit/2396/ -- To view, visit http://gerrit.cloudera.org:8080/3640 To

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

2016-07-13 Thread Dan Burkert (Code Review)
Dan Burkert has posted comments on this change. Change subject: Fix encoding-test on OS X .. Patch Set 2: Code-Review+2 -- To view, visit http://gerrit.cloudera.org:8080/3640 To unsubscribe, visit

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

2016-07-13 Thread Jean-Daniel Cryans (Code Review)
Jean-Daniel Cryans has posted comments on this change. Change subject: KUDU-1487 Add instructions for cutting a release .. Patch Set 3: (1 comment) http://gerrit.cloudera.org:8080/#/c/3614/3/RELEASING.adoc File

[kudu-CR] [java client] Redo how we manage exceptions

2016-07-13 Thread Adar Dembo (Code Review)
Adar Dembo has posted comments on this change. Change subject: [java client] Redo how we manage exceptions .. Patch Set 7: (3 comments) Only a few nits left. When you're done, could you make a pass over the client-consuming

[kudu-CR] [java client] Redo how we manage exceptions

2016-07-13 Thread Kudu Jenkins (Code Review)
Kudu Jenkins has posted comments on this change. Change subject: [java client] Redo how we manage exceptions .. Patch Set 8: Build Started http://104.196.14.100/job/kudu-gerrit/2412/ -- To view, visit

[kudu-CR] [java client] Redo how we manage exceptions

2016-07-13 Thread Adar Dembo (Code Review)
Adar Dembo has posted comments on this change. Change subject: [java client] Redo how we manage exceptions .. Patch Set 8: Code-Review+1 Will leave the +2ing for Dan, since he had some comments originally. -- To view, visit

[kudu-CR] KUDU-1303 Document code style guidelines for C++11 move semantics and rvalue references

2016-07-13 Thread Kudu Jenkins (Code Review)
Kudu Jenkins has posted comments on this change. Change subject: KUDU-1303 Document code style guidelines for C++11 move semantics and rvalue references .. Patch Set 1: Build Started

[kudu-CR] [java client] Redo how we manage exceptions

2016-07-13 Thread Jean-Daniel Cryans (Code Review)
Jean-Daniel Cryans has posted comments on this change. Change subject: [java client] Redo how we manage exceptions .. Patch Set 5: (3 comments)

[kudu-CR] [java client] Redo how we manage exceptions

2016-07-13 Thread Adar Dembo (Code Review)
Adar Dembo has posted comments on this change. Change subject: [java client] Redo how we manage exceptions .. Patch Set 5: (2 comments)

[kudu-CR] rw mutex: add configurable priority

2016-07-13 Thread Adar Dembo (Code Review)
Adar Dembo has posted comments on this change. Change subject: rw_mutex: add configurable priority .. Patch Set 3: > I have experienced in the past an issue where the fairness policy > causes a deadlock. See >

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

2016-07-13 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 2: (1 comment) http://gerrit.cloudera.org:8080/#/c/3631/2//COMMIT_MSG Commit Message: PS2,

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

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

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

2016-07-13 Thread Jean-Daniel Cryans (Code Review)
Hello Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/3631 to look at the new patch set (#3). Change subject: [java client] Integrate with the replay cache .. [java client]

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

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

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

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

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

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

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

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

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

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

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

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

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

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

[kudu-CR] rw mutex: prevent recursive use

2016-07-13 Thread Adar Dembo (Code Review)
Adar Dembo has uploaded a new change for review. http://gerrit.cloudera.org:8080/3641 Change subject: rw_mutex: prevent recursive use .. rw_mutex: prevent recursive use Todd provided an example[1] of deadlocked rwlocks due

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

2016-07-13 Thread Adar Dembo (Code Review)
Adar Dembo has uploaded a new change for review. http://gerrit.cloudera.org:8080/3643 Change subject: KUDU-1374: send full tablet report when new leader master is detected .. KUDU-1374: send full tablet report when new

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

2016-07-13 Thread Adar Dembo (Code Review)
Adar Dembo has uploaded a new change for review. http://gerrit.cloudera.org:8080/3642 Change subject: master: add assert checks for leader_lock .. master: add assert checks for leader_lock A side effect of recursive checking

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

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

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

2016-07-13 Thread Adar Dembo (Code Review)
Adar Dembo has uploaded a new change for review. http://gerrit.cloudera.org:8080/3645 Change subject: master: do not delete unknown tablets .. master: do not delete unknown tablets Quoting from the multi-master design doc:

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

2016-07-13 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 (#5). Change subject: KUDU-1358 (part 2): heartbeat to every master .. KUDU-1358

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

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

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

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

[kudu-CR] rw mutex: prevent recursive use

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

[kudu-CR] client.h: doxygen comments for C++ API

2016-07-13 Thread Alexey Serbin (Code Review)
Alexey Serbin has posted comments on this change. Change subject: client.h: doxygen comments for C++ API .. Patch Set 5: (9 comments) Thank you for the review! I took a look at vera++ as a candidate for syntax rule/style

[kudu-CR] KUDU-1517 Implement doc feedback from Sue M

2016-07-13 Thread Kudu Jenkins (Code Review)
Kudu Jenkins has posted comments on this change. Change subject: KUDU-1517 Implement doc feedback from Sue M .. Patch Set 1: Build Started http://104.196.14.100/job/kudu-gerrit/2391/ -- To view, visit

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

2016-07-13 Thread Misty Stanley-Jones (Code Review)
Misty Stanley-Jones has posted comments on this change. Change subject: KUDU-1487 Add instructions for cutting a release .. Patch Set 1: (1 comment) http://gerrit.cloudera.org:8080/#/c/3614/1/RELEASING.adoc File

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

2016-07-13 Thread Dan Burkert (Code Review)
Dan Burkert has posted comments on this change. Change subject: [java client] Integrate with the replay cache .. Patch Set 2: (1 comment) http://gerrit.cloudera.org:8080/#/c/3631/2//COMMIT_MSG Commit Message: PS2, Line 12:

[kudu-CR] [java client] Redo how we manage exceptions

2016-07-13 Thread Jean-Daniel Cryans (Code Review)
Jean-Daniel Cryans has posted comments on this change. Change subject: [java client] Redo how we manage exceptions .. Patch Set 5: (16 comments) > (16 comments) > > Definitely make a pass over the Javadoc for: > 1. Adding

[kudu-CR] client.h: doxygen comments for C++ API

2016-07-13 Thread Alexey Serbin (Code Review)
Alexey Serbin has posted comments on this change. Change subject: client.h: doxygen comments for C++ API .. Patch Set 5: (1 comment) http://gerrit.cloudera.org:8080/#/c/3619/5/CMakeLists.txt File CMakeLists.txt: Line 963:

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

2016-07-13 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 2: (1 comment) http://gerrit.cloudera.org:8080/#/c/3631/2//COMMIT_MSG Commit Message: PS2,

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

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

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

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

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

2016-07-13 Thread Jean-Daniel Cryans (Code Review)
Jean-Daniel Cryans has posted comments on this change. Change subject: KUDU-1487 Add instructions for cutting a release .. Patch Set 1: (1 comment) http://gerrit.cloudera.org:8080/#/c/3614/1/RELEASING.adoc File

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

2016-07-13 Thread David Ribeiro Alves (Code Review)
David Ribeiro Alves has posted comments on this change. Change subject: [java client] Integrate with the replay cache .. Patch Set 2: (1 comment) http://gerrit.cloudera.org:8080/#/c/3631/2//COMMIT_MSG Commit Message: PS2,

[kudu-CR] KUDU-1516 ksck should check for more raft-related status issues (partial)

2016-07-13 Thread Todd Lipcon (Code Review)
Todd Lipcon has posted comments on this change. Change subject: KUDU-1516 ksck should check for more raft-related status issues (partial) .. Patch Set 1: (5 comments) is it possible to add an end-to-end test of this in

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

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

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

2016-07-13 Thread Adar Dembo (Code Review)
Hello Dan Burkert, Todd Lipcon, 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 (#8). Change subject: master: add read-write lock to serialize operations around elections

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

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

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

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

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

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

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

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

[kudu-CR] env: add GetFileSizeOnDiskRecursively

2016-07-13 Thread Kudu Jenkins (Code Review)
Kudu Jenkins has posted comments on this change. Change subject: env: add GetFileSizeOnDiskRecursively .. Patch Set 2: Build Started http://104.196.14.100/job/kudu-gerrit/2416/ -- To view, visit

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

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

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

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

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

2016-07-13 Thread Adar Dembo (Code Review)
Adar Dembo has posted comments on this change. Change subject: master: do not delete unknown tablets .. Patch Set 2: Verified+1 One run failed from the same dist-failure as http://gerrit.cloudera.org:8080/3610/6. Another

[kudu-CR] rw mutex: prevent recursive use

2016-07-13 Thread Todd Lipcon (Code Review)
Todd Lipcon has posted comments on this change. Change subject: rw_mutex: prevent recursive use .. Patch Set 1: (2 comments) http://gerrit.cloudera.org:8080/#/c/3641/1/src/kudu/util/rw_mutex-test.cc File

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

2016-07-13 Thread Misty Stanley-Jones (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 (#4). Change subject: KUDU-1487 Add instructions for cutting a release .. KUDU-1487

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

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

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

2016-07-13 Thread Jean-Daniel Cryans (Code Review)
Jean-Daniel Cryans has posted comments on this change. Change subject: KUDU-1487 Add instructions for cutting a release .. Patch Set 4: (1 comment) http://gerrit.cloudera.org:8080/#/c/3614/4/RELEASING.adoc File

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

2016-07-13 Thread Misty Stanley-Jones (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 (#5). Change subject: KUDU-1487 Add instructions for cutting a release .. KUDU-1487

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

2016-07-13 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 4: (2 comments) Added ttl config to master and a TTL expiry test, as well as a fix for ksck

[kudu-CR] env: add GetFileSizeOnDiskRecursively

2016-07-13 Thread Adar Dembo (Code Review)
Adar Dembo has posted comments on this change. Change subject: env: add GetFileSizeOnDiskRecursively .. Patch Set 2: Verified+1 Unrelated flake in ClientStressTest.StartScans. -- To view, visit

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

2016-07-13 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 6: Verified+1 Looks like I got hit by some sort of sporadic dist-test failure: open: Text file busy

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

2016-07-13 Thread Misty Stanley-Jones (Code Review)
Misty Stanley-Jones has posted comments on this change. Change subject: KUDU-1487 Add instructions for cutting a release .. Patch Set 4: (1 comment) http://gerrit.cloudera.org:8080/#/c/3614/4/RELEASING.adoc File

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

2016-07-13 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 6: Verified+1 Two isolate failures and one chrpath failure. -- To view, visit

[kudu-CR] rw mutex: add configurable priority

2016-07-13 Thread Todd Lipcon (Code Review)
Todd Lipcon 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] KUDU-1311 [master] support adding and dropping range partitions

2016-07-13 Thread Dan Burkert (Code Review)
Dan Burkert has uploaded a new change for review. http://gerrit.cloudera.org:8080/3648 Change subject: KUDU-1311 [master] support adding and dropping range partitions .. KUDU-1311 [master] support adding and dropping range

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

2016-07-13 Thread Dan Burkert (Code Review)
Hello Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/3581 to look at the new patch set (#4). Change subject: [c++-client]: cache non-covering ranges in meta cache ..

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

2016-07-13 Thread Kudu Jenkins (Code Review)
Kudu Jenkins has posted comments on this change. Change subject: [c++-client]: cache non-covering ranges in meta cache .. Patch Set 4: Build Started http://104.196.14.100/job/kudu-gerrit/2428/ -- To view, visit

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

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

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

2016-07-13 Thread Kudu Jenkins (Code Review)
Kudu Jenkins has posted comments on this change. Change subject: Fix encoding-test on OS X .. Patch Set 2: -Verified Build Started http://104.196.14.100/job/kudu-gerrit/2426/ -- To view, visit

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

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

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

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

[kudu-CR] Integrate the result tracker with writes

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

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

2016-07-13 Thread Kudu Jenkins (Code Review)
Kudu Jenkins 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: Build Started http://104.196.14.100/job/kudu-gerrit/2380/ --

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

2016-07-13 Thread David Ribeiro Alves (Code Review)
Hello 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 (#15). Change subject: Disable exactly once semantics by default and add a flag to enable it for tests

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

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

[kudu-CR] WIP: Memory tracking for result tracker

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