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

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

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

2016-07-19 Thread Alexey Serbin (Code Review)
Hello Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/3619 to look at the new patch set (#12). Change subject: Doxygen for C++ client API .. Doxygen for C++ client API If

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

2016-07-19 Thread Alexey Serbin (Code Review)
Alexey Serbin has posted comments on this change. Change subject: Doxygen for C++ client API .. Patch Set 9: (5 comments) http://gerrit.cloudera.org:8080/#/c/3619/9/src/kudu/client/client.h File src/kudu/client/client.h: >

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

2016-07-19 Thread Adar Dembo (Code Review)
Adar Dembo has uploaded a new patch set (#10). Change subject: KUDU-1358 (part 3): new multi-master stress test .. KUDU-1358 (part 3): new multi-master stress test This commit adds a stress test for multiple masters. The idea

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

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

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

2016-07-19 Thread Adar Dembo (Code Review)
Adar Dembo has uploaded a new patch set (#6). Change subject: master: do not delete unknown tablets .. master: do not delete unknown tablets Quoting from docs/design-docs/multi-master-1.0.md: "The master and/or tserver must

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

2016-07-19 Thread Adar Dembo (Code Review)
Hello Dan Burkert, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/3609 to look at the new patch set (#10). Change subject: KUDU-1358 (part 1): master should accept heartbeat even if follower

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

2016-07-19 Thread Adar Dembo (Code Review)
Adar Dembo has uploaded a new patch set (#6). Change subject: KUDU-1374: send full tablet report when new leader master is detected .. KUDU-1374: send full tablet report when new leader master is detected This should help

[kudu-CR] master: additional leader lock assertions in catalog manager

2016-07-19 Thread Kudu Jenkins (Code Review)
Kudu Jenkins has posted comments on this change. Change subject: master: additional leader lock assertions in catalog manager .. Patch Set 1: Build Started http://104.196.14.100/job/kudu-gerrit/2565/ -- To view, visit

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

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

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

2016-07-19 Thread Adar Dembo (Code Review)
Adar Dembo has posted comments on this change. Change subject: KUDU-1374: send full tablet report when new leader master is detected .. Patch Set 5: (3 comments) http://gerrit.cloudera.org:8080/#/c/3643/5//COMMIT_MSG Commit

[kudu-CR] tablet peer: update status message on failure to start

2016-07-19 Thread Jean-Daniel Cryans (Code Review)
Jean-Daniel Cryans has posted comments on this change. Change subject: tablet_peer: update status message on failure to start .. Patch Set 1: Code-Review+2 -- To view, visit http://gerrit.cloudera.org:8080/3682 To

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

2016-07-19 Thread Jean-Daniel Cryans (Code Review)
Jean-Daniel Cryans has posted comments on this change. Change subject: [java-client] Re-enable multi-master tests .. Patch Set 4: -Verified Actually, holding off the +1 so that we don't push this patch before the other

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

2016-07-19 Thread Jean-Daniel Cryans (Code Review)
Jean-Daniel Cryans has posted comments on this change. Change subject: [java-client] Re-enable multi-master tests .. Patch Set 4: Verified+1 ClientStressTest.TestStartScans is flaky. -- To view, visit

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

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

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

2016-07-19 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 5: (2 comments)

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

2016-07-19 Thread Adar Dembo (Code Review)
Adar Dembo has posted comments on this change. Change subject: [java-client] Re-enable multi-master tests .. Patch Set 4: Code-Review+2 -- To view, visit http://gerrit.cloudera.org:8080/3654 To unsubscribe, visit

[kudu-CR] KUDU-1416 Upsert support for Flume sink

2016-07-19 Thread Mike Percy (Code Review)
Mike Percy has posted comments on this change. Change subject: KUDU-1416 Upsert support for Flume sink .. Patch Set 1: Yeah, just needs another rev -- To view, visit http://gerrit.cloudera.org:8080/3157 To unsubscribe, visit

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

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

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

2016-07-19 Thread Jean-Daniel Cryans (Code Review)
Jean-Daniel Cryans has uploaded a new patch set (#4). Change subject: [java-client] Re-enable multi-master tests .. [java-client] Re-enable multi-master tests This patch makes TestMasterFailover useful again. It also adds the

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

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

[kudu-CR] More disk reservation-itest flaky test workarounds

2016-07-19 Thread Mike Percy (Code Review)
Mike Percy has submitted this change and it was merged. Change subject: More disk_reservation-itest flaky test workarounds .. More disk_reservation-itest flaky test workarounds Change-Id:

[kudu-CR] More disk reservation-itest flaky test workarounds

2016-07-19 Thread Mike Percy (Code Review)
Mike Percy has uploaded a new change for review. http://gerrit.cloudera.org:8080/3683 Change subject: More disk_reservation-itest flaky test workarounds .. More disk_reservation-itest flaky test workarounds Change-Id:

[kudu-CR] More disk reservation-itest flaky test workarounds

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

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

2016-07-19 Thread Adar Dembo (Code Review)
Adar Dembo has posted comments on this change. Change subject: master: do not delete unknown tablets .. Patch Set 5: (3 comments) http://gerrit.cloudera.org:8080/#/c/3645/5//COMMIT_MSG Commit Message: Line 7: master: do not

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

2016-07-19 Thread Adar Dembo (Code Review)
Adar Dembo has posted comments on this change. Change subject: Doxygen for C++ client API .. Patch Set 11: (5 comments) http://gerrit.cloudera.org:8080/#/c/3619/9/src/kudu/client/client.h File src/kudu/client/client.h: > 1.

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

2016-07-19 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 2: here's some example output on a cluster with a messed up table:

[kudu-CR] tablet peer: update status message on failure to start

2016-07-19 Thread Todd Lipcon (Code Review)
Hello Jean-Daniel Cryans, Mike Percy, Will Berkeley, I'd like you to do a code review. Please visit http://gerrit.cloudera.org:8080/3682 to review the following change. Change subject: tablet_peer: update status message on failure to start

[kudu-CR] tablet peer: update status message on failure to start

2016-07-19 Thread Kudu Jenkins (Code Review)
Kudu Jenkins has posted comments on this change. Change subject: tablet_peer: update status message on failure to start .. Patch Set 1: Build Started http://104.196.14.100/job/kudu-gerrit/2556/ -- To view, visit

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

2016-07-19 Thread Todd Lipcon (Code Review)
Hello Jean-Daniel Cryans, Mike Percy, Will Berkeley, Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/3632 to look at the new patch set (#2). Change subject: KUDU-1516 ksck should check for more raft-related status issues (partial)

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

2016-07-19 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 2: Build Started http://104.196.14.100/job/kudu-gerrit/2555/ -- To

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

2016-07-19 Thread Adar Dembo (Code Review)
Adar Dembo has posted comments on this change. Change subject: KUDU-1311 [master] support adding and dropping range partitions .. Patch Set 8: (2 comments)

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

2016-07-19 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: (4 comments)

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

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

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

2016-07-19 Thread Dan Burkert (Code Review)
Hello Adar Dembo, 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 (#8). Change subject: KUDU-1311 [master] support adding and dropping range partitions

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

2016-07-19 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: (4 comments) http://gerrit.cloudera.org:8080/#/c/3610/9/src/kudu/tserver/heartbeater.cc File

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

2016-07-19 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: (1 comment) > (1 comment) > > Any way to system test this, like calling

[kudu-CR] C++ client: fix on KuduSession::GetPendingErrors()

2016-07-19 Thread Todd Lipcon (Code Review)
Todd Lipcon has posted comments on this change. Change subject: C++ client: fix on KuduSession::GetPendingErrors() .. Patch Set 1: Code-Review+2 -- To view, visit http://gerrit.cloudera.org:8080/3677 To unsubscribe, visit

[kudu-CR] C++ client: fix on KuduSession::GetPendingErrors()

2016-07-19 Thread Kudu Jenkins (Code Review)
Kudu Jenkins has posted comments on this change. Change subject: C++ client: fix on KuduSession::GetPendingErrors() .. Patch Set 1: Build Started http://104.196.14.100/job/kudu-gerrit/2553/ -- To view, visit

[kudu-CR] C++ client: fix on KuduSession::GetPendingErrors()

2016-07-19 Thread Alexey Serbin (Code Review)
Alexey Serbin has uploaded a new change for review. http://gerrit.cloudera.org:8080/3677 Change subject: C++ client: fix on KuduSession::GetPendingErrors() .. C++ client: fix on KuduSession::GetPendingErrors() If re-using

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

2016-07-19 Thread Dan Burkert (Code Review)
Dan Burkert has posted comments on this change. Change subject: [java client] Integrate with the replay cache .. Patch Set 5: (2 comments)

[kudu-CR] Add time/watermark based garbage collection to ResultTracker

2016-07-19 Thread Alexey Serbin (Code Review)
Alexey Serbin has posted comments on this change. Change subject: Add time/watermark based garbage collection to ResultTracker .. Patch Set 13: (11 comments)

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

2016-07-19 Thread Jean-Daniel Cryans (Code Review)
Hello Adar Dembo, 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 (#9). Change subject: [java client] Redo how we manage exceptions ..

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

2016-07-19 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 8: (3 comments)

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

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

[kudu-CR] [java-client] fix soundness hole in flushing async kudu session

2016-07-19 Thread Dan Burkert (Code Review)
Dan Burkert has submitted this change and it was merged. Change subject: [java-client] fix soundness hole in flushing async kudu session .. [java-client] fix soundness hole in flushing async kudu session This fixes an issue

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

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

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

2016-07-19 Thread Alexey Serbin (Code Review)
Hello Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/3619 to look at the new patch set (#11). Change subject: Doxygen for C++ client API .. Doxygen for C++ client API If

[kudu-CR] [java-client] fix soundness hole in flushing async kudu session

2016-07-19 Thread Jean-Daniel Cryans (Code Review)
Jean-Daniel Cryans has posted comments on this change. Change subject: [java-client] fix soundness hole in flushing async kudu session .. Patch Set 1: Code-Review+2 -- To view, visit http://gerrit.cloudera.org:8080/3676 To

[kudu-CR] [java-client] fix soundness hole in flushing async kudu session

2016-07-19 Thread Adar Dembo (Code Review)
Adar Dembo has posted comments on this change. Change subject: [java-client] fix soundness hole in flushing async kudu session .. Patch Set 1: Code-Review+2 -- To view, visit http://gerrit.cloudera.org:8080/3676 To

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

2016-07-19 Thread Adar Dembo (Code Review)
Adar Dembo has posted comments on this change. Change subject: KUDU-1311 [master] support adding and dropping range partitions .. Patch Set 7: Code-Review+2 -- To view, visit http://gerrit.cloudera.org:8080/3648 To

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

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

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

2016-07-19 Thread Alexey Serbin (Code Review)
Alexey Serbin has posted comments on this change. Change subject: doxygen for C++ client API .. Patch Set 9: (64 comments) http://gerrit.cloudera.org:8080/#/c/3619/9/src/kudu/client/client.h File src/kudu/client/client.h:

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

2016-07-19 Thread Alexey Serbin (Code Review)
Hello Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/3619 to look at the new patch set (#10). Change subject: doxygen for C++ client API .. doxygen for C++ client API If

[kudu-CR] [java-client] fix soundness hole in flushing async kudu session

2016-07-19 Thread Dan Burkert (Code Review)
Hello Adar Dembo, I'd like you to do a code review. Please visit http://gerrit.cloudera.org:8080/3676 to review the following change. Change subject: [java-client] fix soundness hole in flushing async kudu session ..

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

2016-07-19 Thread Dan Burkert (Code Review)
Dan Burkert has posted comments on this change. Change subject: [java client] Redo how we manage exceptions .. Patch Set 8: (3 comments) Looking great overall.

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

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

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

2016-07-19 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 6: (10 comments)

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

2016-07-19 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 (#7). Change subject: KUDU-1311 [master] support adding and dropping range partitions

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

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

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

2016-07-19 Thread Dan Burkert (Code Review)
Dan Burkert has submitted this change and it was merged. Change subject: [c++-client]: cache non-covering ranges in meta cache .. [c++-client]: cache non-covering ranges in meta cache This commit introduces a few features to

[kudu-CR] [c++-client] add LESS and GREATER column predicates

2016-07-19 Thread Dan Burkert (Code Review)
Dan Burkert has submitted this change and it was merged. Change subject: [c++-client] add LESS and GREATER column predicates .. [c++-client] add LESS and GREATER column predicates Change-Id:

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

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

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

2016-07-19 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 (#5). Change subject: [java client] Integrate with the replay cache .. [java client]

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

2016-07-19 Thread Jean-Daniel Cryans (Code Review)
Jean-Daniel Cryans has posted comments on this change. Change subject: [java-client] Re-enable multi-master tests .. Patch Set 3: Verified+1 So much flakiness right now on jenkins. -- To view, visit

[kudu-CR] [c++-client] add LESS and GREATER column predicates

2016-07-19 Thread Dan Burkert (Code Review)
Hello Adar Dembo, Todd Lipcon, Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/3674 to look at the new patch set (#2). Change subject: [c++-client] add LESS and GREATER column predicates

[kudu-CR] [c++-client] add LESS and GREATER column predicates

2016-07-19 Thread Kudu Jenkins (Code Review)
Kudu Jenkins has posted comments on this change. Change subject: [c++-client] add LESS and GREATER column predicates .. Patch Set 2: Build Started http://104.196.14.100/job/kudu-gerrit/2546/ -- To view, visit

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

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

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

2016-07-19 Thread Jean-Daniel Cryans (Code Review)
Hello Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/3654 to look at the new patch set (#3). Change subject: [java-client] Re-enable multi-master tests .. [java-client]

[kudu-CR](gh-pages) Clean up community web page and add commits@ mailing list

2016-07-19 Thread Mike Percy (Code Review)
Mike Percy has submitted this change and it was merged. Change subject: Clean up community web page and add commits@ mailing list .. Clean up community web page and add commits@ mailing list Change-Id:

[kudu-CR](gh-pages) Add dropdown menu for Community nav button

2016-07-19 Thread Mike Percy (Code Review)
Mike Percy has submitted this change and it was merged. Change subject: Add dropdown menu for Community nav button .. Add dropdown menu for Community nav button * The dropdown is disabled on very small screens (mobile) * The

[kudu-CR](gh-pages) Clean up community web page and add commits@ mailing list

2016-07-19 Thread Mike Percy (Code Review)
Mike Percy has posted comments on this change. Change subject: Clean up community web page and add commits@ mailing list .. Patch Set 3: Verified+1 -- To view, visit http://gerrit.cloudera.org:8080/3666 To unsubscribe, visit

[kudu-CR](gh-pages) Add dropdown menu for Community nav button

2016-07-19 Thread Mike Percy (Code Review)
Mike Percy has posted comments on this change. Change subject: Add dropdown menu for Community nav button .. Patch Set 4: Verified+1 -- To view, visit http://gerrit.cloudera.org:8080/3665 To unsubscribe, visit

[kudu-CR](gh-pages) Clean up community web page and add commits@ mailing list

2016-07-19 Thread Todd Lipcon (Code Review)
Todd Lipcon has posted comments on this change. Change subject: Clean up community web page and add commits@ mailing list .. Patch Set 3: Code-Review+2 -- To view, visit http://gerrit.cloudera.org:8080/3666 To unsubscribe,

[kudu-CR](gh-pages) Add dropdown menu for Community nav button

2016-07-19 Thread Todd Lipcon (Code Review)
Todd Lipcon has posted comments on this change. Change subject: Add dropdown menu for Community nav button .. Patch Set 4: Code-Review+2 -- To view, visit http://gerrit.cloudera.org:8080/3665 To unsubscribe, visit

[kudu-CR] KUDU-1416 Upsert support for Flume sink

2016-07-19 Thread Todd Lipcon (Code Review)
Todd Lipcon has posted comments on this change. Change subject: KUDU-1416 Upsert support for Flume sink .. Patch Set 1: What's the story on this one? Needs another rev, right? -- To view, visit

[kudu-CR] KUDU-1492: Show column encodings/compression on table page in master

2016-07-19 Thread Dinesh Bhat (Code Review)
Dinesh Bhat has posted comments on this change. Change subject: KUDU-1492: Show column encodings/compression on table page in master .. Patch Set 2: (6 comments) http://gerrit.cloudera.org:8080/#/c/3667/1//COMMIT_MSG Commit

[kudu-CR] KUDU-1492: Show column encodings/compression on table page in master

2016-07-19 Thread Dinesh Bhat (Code Review)
Hello Mike Percy, Alexey Serbin, Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/3667 to look at the new patch set (#2). Change subject: KUDU-1492: Show column encodings/compression on table page in master

[kudu-CR] Attempt to unflake disk reservation-itest a little more

2016-07-19 Thread Mike Percy (Code Review)
Mike Percy has posted comments on this change. Change subject: Attempt to unflake disk_reservation-itest a little more .. Patch Set 1: Code-Review+2 Verified+1 Pushing this trivial change through, would like to see if it is

[kudu-CR] Attempt to unflake disk reservation-itest a little more

2016-07-19 Thread Kudu Jenkins (Code Review)
Kudu Jenkins has posted comments on this change. Change subject: Attempt to unflake disk_reservation-itest a little more .. Patch Set 1: Build Started http://104.196.14.100/job/kudu-gerrit/2544/ -- To view, visit

[kudu-CR] Attempt to unflake disk reservation-itest a little more

2016-07-19 Thread Mike Percy (Code Review)
Mike Percy has uploaded a new change for review. http://gerrit.cloudera.org:8080/3675 Change subject: Attempt to unflake disk_reservation-itest a little more .. Attempt to unflake disk_reservation-itest a little more

[kudu-CR] KUDU-1492: Show column encodings/compression on table page in master

2016-07-19 Thread Kudu Jenkins (Code Review)
Kudu Jenkins has posted comments on this change. Change subject: KUDU-1492: Show column encodings/compression on table page in master .. Patch Set 2: Build Started http://104.196.14.100/job/kudu-gerrit/2543/ -- To view,

[kudu-CR] [c++-client] add LESS and GREATER column predicates

2016-07-19 Thread Todd Lipcon (Code Review)
Todd Lipcon has posted comments on this change. Change subject: [c++-client] add LESS and GREATER column predicates .. Patch Set 1: Code-Review+2 (1 comment)

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

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

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

2016-07-19 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 9: Code-Review+2 (1 comment)

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

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

[kudu-CR] Memory tracking for result tracker

2016-07-19 Thread David Ribeiro Alves (Code Review)
David Ribeiro Alves has posted comments on this change. Change subject: Memory tracking for result tracker .. Patch Set 19: theres a bunch of spurious failures, mostly on lint builds. trying once more before overriding --

[kudu-CR] Memory tracking for result tracker

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

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

2016-07-19 Thread Will Berkeley (Code Review)
Will Berkeley has posted comments on this change. Change subject: KUDU-1516 ksck should check for more raft-related status issues (partial) .. Patch Set 1: > BTW, I tried this on a cluster with a bad table: > WARNING: Unable

[kudu-CR] Add time/watermark based garbage collection to ResultTracker

2016-07-19 Thread Kudu Jenkins (Code Review)
Kudu Jenkins has posted comments on this change. Change subject: Add time/watermark based garbage collection to ResultTracker .. Patch Set 13: Build Started http://104.196.14.100/job/kudu-gerrit/2537/ -- To view, visit

[kudu-CR] Add time/watermark based garbage collection to ResultTracker

2016-07-19 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 (#13). Change subject: Add time/watermark based garbage collection to ResultTracker ..

[kudu-CR] [c++-client] add LESS and GREATER column predicates

2016-07-19 Thread Dan Burkert (Code Review)
Hello Adar Dembo, Todd Lipcon, I'd like you to do a code review. Please visit http://gerrit.cloudera.org:8080/3674 to review the following change. Change subject: [c++-client] add LESS and GREATER column predicates ..

[kudu-CR] [c++-client] add LESS and GREATER column predicates

2016-07-19 Thread Kudu Jenkins (Code Review)
Kudu Jenkins has posted comments on this change. Change subject: [c++-client] add LESS and GREATER column predicates .. Patch Set 1: Build Started http://104.196.14.100/job/kudu-gerrit/2536/ -- To view, visit

[kudu-CR] KUDU-236. Implement tablet history GC

2016-07-19 Thread Mike Percy (Code Review)
Mike Percy has posted comments on this change. Change subject: KUDU-236. Implement tablet history GC .. Patch Set 4: Weird... RELEASE build failed in patchset 3 with a thirdparty build error. It's like the disk disappeared:

[kudu-CR] KUDU-236. Implement tablet history GC

2016-07-19 Thread Kudu Jenkins (Code Review)
Kudu Jenkins has posted comments on this change. Change subject: KUDU-236. Implement tablet history GC .. Patch Set 4: Build Started http://104.196.14.100/job/kudu-gerrit/2535/ -- To view, visit

[kudu-CR] KUDU-236. Implement tablet history GC

2016-07-19 Thread Mike Percy (Code Review)
Hello Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/3076 to look at the new patch set (#4). Change subject: KUDU-236. Implement tablet history GC .. KUDU-236. Implement

[kudu-CR](gh-pages) Add dropdown menu for Community nav button

2016-07-19 Thread Mike Percy (Code Review)
Mike Percy has posted comments on this change. Change subject: Add dropdown menu for Community nav button .. Patch Set 4: OK I fixed the iPad issue. Tested on an iPad 3. -- To view, visit http://gerrit.cloudera.org:8080/3665

[kudu-CR](gh-pages) Add dropdown menu for Community nav button

2016-07-19 Thread Mike Percy (Code Review)
Mike Percy has uploaded a new patch set (#4). Change subject: Add dropdown menu for Community nav button .. Add dropdown menu for Community nav button * The dropdown is disabled on very small screens (mobile) * The dropdown is

[kudu-CR] KUDU-236. Implement tablet history GC

2016-07-19 Thread Kudu Jenkins (Code Review)
Kudu Jenkins has posted comments on this change. Change subject: KUDU-236. Implement tablet history GC .. Patch Set 3: Build Started http://104.196.14.100/job/kudu-gerrit/2534/ -- To view, visit

[kudu-CR] KUDU-236. Implement tablet history GC

2016-07-19 Thread Mike Percy (Code Review)
Hello Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/3076 to look at the new patch set (#3). Change subject: KUDU-236. Implement tablet history GC .. KUDU-236. Implement

  1   2   >