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

2016-07-21 Thread Jean-Daniel Cryans (Code Review)
Jean-Daniel Cryans has submitted this change and it was merged. Change subject: [java client] Redo how we manage exceptions .. [java client] Redo how we manage exceptions Right now the exceptions are hard to handle in the Java

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

2016-07-21 Thread Jean-Daniel Cryans (Code Review)
Jean-Daniel Cryans has submitted this change and it was merged. Change subject: [java client] Integrate with the replay cache .. [java client] Integrate with the replay cache This patch adds the required functionality to have t

[kudu-CR] ksck: improve filtering capability

2016-07-20 Thread Jean-Daniel Cryans (Code Review)
Jean-Daniel Cryans has posted comments on this change. Change subject: ksck: improve filtering capability .. Patch Set 1: Code-Review+2 -- To view, visit http://gerrit.cloudera.org:8080/3716 To unsubscribe, visit http://gerrit

[kudu-CR] ksck: improve output for long-running ksck checksums

2016-07-20 Thread Jean-Daniel Cryans (Code Review)
Jean-Daniel Cryans has posted comments on this change. Change subject: ksck: improve output for long-running ksck checksums .. Patch Set 1: Code-Review+2 (1 comment) http://gerrit.cloudera.org:8080/#/c/3715/1//COMMIT_MSG Commi

[kudu-CR] ksck: fix a crash in checksum mode on tables with many tablets

2016-07-20 Thread Jean-Daniel Cryans (Code Review)
Jean-Daniel Cryans has posted comments on this change. Change subject: ksck: fix a crash in checksum mode on tables with many tablets .. Patch Set 1: Code-Review+2 -- To view, visit http://gerrit.cloudera.org:8080/3714 To unsu

[kudu-CR] ksck: also print info about the good replicas for a bad tablet

2016-07-20 Thread Jean-Daniel Cryans (Code Review)
Jean-Daniel Cryans has posted comments on this change. Change subject: ksck: also print info about the good replicas for a bad tablet .. Patch Set 1: Code-Review+2 -- To view, visit http://gerrit.cloudera.org:8080/3706 To unsu

[kudu-CR] ksck: multi-thread the fetching of replica info from tablet servers

2016-07-20 Thread Jean-Daniel Cryans (Code Review)
Jean-Daniel Cryans has posted comments on this change. Change subject: ksck: multi-thread the fetching of replica info from tablet servers .. Patch Set 2: Code-Review+2 -- To view, visit http://gerrit.cloudera.org:8080/3705 T

[kudu-CR] ksck: report hostnames instead of IP addresses for tablet servers

2016-07-20 Thread Jean-Daniel Cryans (Code Review)
Jean-Daniel Cryans has posted comments on this change. Change subject: ksck: report hostnames instead of IP addresses for tablet servers .. Patch Set 2: Code-Review+2 -- To view, visit http://gerrit.cloudera.org:8080/3704 To

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

2016-07-20 Thread Jean-Daniel Cryans (Code Review)
Jean-Daniel Cryans has posted comments on this change. Change subject: KUDU-1516 ksck should check for more raft-related status issues (partial) .. Patch Set 4: (3 comments) http://gerrit.cloudera.org:8080/#/c/3632/4/src/kudu

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

2016-07-20 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 (#12). Change subject: [java client] Redo how we manage exceptions .. [jav

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

2016-07-20 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 11: (1 comment) http://gerrit.cloudera.org:8080/#/c/3055/11/java/kudu-client/src/main/java/org/kud

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

2016-07-20 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 (#10). Change subject: [java client] Redo how we manage exceptions .. [jav

[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 unsubscribe,

[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 patches

[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 http://gerrit.cloudera.o

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

[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) http://gerrit.cloudera.org:8080/#/c/3631/5/java/kudu-client/src/main/java/org/ku

[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 ki

[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) http://gerrit.cloudera.org:8080/#/c/3654/3/java/kudu-client/src/main/java/org/kudud

[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 .. [java

[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) http://gerrit.cloudera.org:8080/#/c/3055/8/java/kudu-client/src/main/java/org/kudu

[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 uns

[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] I

[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 http://gerrit.cloudera.org

[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] Re-e

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

2016-07-18 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 (#2). Change subject: WIP [java-client] Re-enable multi-master tests .. WIP [java-clie

[kudu-CR](gh-pages) Add weekly update for 7/18

2016-07-18 Thread Jean-Daniel Cryans (Code Review)
Jean-Daniel Cryans has posted comments on this change. Change subject: Add weekly update for 7/18 .. Patch Set 1: Code-Review+2 -- To view, visit http://gerrit.cloudera.org:8080/3671 To unsubscribe, visit http://gerrit.clouder

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

2016-07-18 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: > hm, what's the status on this. Still waiting on Dan? Yeah I told Dan that helping out Adar fi

[kudu-CR] Add port in web ui tables, add role and table name to /tablet page

2016-07-15 Thread Jean-Daniel Cryans (Code Review)
Jean-Daniel Cryans has submitted this change and it was merged. Change subject: Add port in web ui tables, add role and table name to /tablet page .. Add port in web ui tables, add role and table name to /tablet page Couple of

[kudu-CR] Add port in web ui tables, add role and table name to /tablet page

2016-07-15 Thread Jean-Daniel Cryans (Code Review)
Jean-Daniel Cryans has posted comments on this change. Change subject: Add port in web ui tables, add role and table name to /tablet page .. Patch Set 1: Code-Review+2 -- To view, visit http://gerrit.cloudera.org:8080/3553 To

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

2016-07-15 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: (1 comment) http://gerrit.cloudera.org:8080/#/c/3628/5/src/kudu/rpc/result_tracker.cc File src

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

2016-07-15 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: (3 comments) http://gerrit.cloudera.org:8080/#/c/3631/4/java/kudu-client/src/main/java/org/ku

[kudu-CR] Move the maintenance manager to util

2016-07-14 Thread Jean-Daniel Cryans (Code Review)
Jean-Daniel Cryans has posted comments on this change. Change subject: Move the maintenance manager to util .. Patch Set 3: Code-Review+2 -- To view, visit http://gerrit.cloudera.org:8080/3656 To unsubscribe, visit http://gerr

[kudu-CR] Integrate the result tracker with writes

2016-07-14 Thread Jean-Daniel Cryans (Code Review)
Jean-Daniel Cryans has posted comments on this change. Change subject: Integrate the result tracker with writes .. Patch Set 28: Code-Review+2 -- To view, visit http://gerrit.cloudera.org:8080/3449 To unsubscribe, visit http:/

[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: (2 comments) http://gerrit.cloudera.org:8080/#/c/3628/5/src/kudu/rpc/result_tracker.cc File sr

[kudu-CR] Move the maintenance manager to util

2016-07-14 Thread Jean-Daniel Cryans (Code Review)
Jean-Daniel Cryans has posted comments on this change. Change subject: Move the maintenance manager to util .. Patch Set 2: (1 comment) http://gerrit.cloudera.org:8080/#/c/3656/2/src/kudu/tablet/CMakeLists.txt File src/kudu/ta

[kudu-CR] Integrate the result tracker with writes

2016-07-14 Thread Jean-Daniel Cryans (Code Review)
Jean-Daniel Cryans has posted comments on this change. Change subject: Integrate the result tracker with writes .. Patch Set 27: Code-Review+2 -- To view, visit http://gerrit.cloudera.org:8080/3449 To unsubscribe, visit http:/

[kudu-CR] Integrate the result tracker with writes

2016-07-14 Thread Jean-Daniel Cryans (Code Review)
Jean-Daniel Cryans has posted comments on this change. Change subject: Integrate the result tracker with writes .. Patch Set 26: (2 comments) http://gerrit.cloudera.org:8080/#/c/3449/26/src/kudu/tablet/transactions/write_trans

[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) http://gerrit.cloudera.org:8080/#/c/3631/4/java/kudu-client/src/main/java/org/kud

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

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/3631 to look at the new patch set (#4). Change subject: [java client] Integrate with the replay cache .. [java client] I

[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 submitted this change and it was merged. 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 Change-Id: If8aa4a73ef33c413a52c5cb6

[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: client

[kudu-CR] Move the maintenance manager to util

2016-07-14 Thread Jean-Daniel Cryans (Code Review)
Jean-Daniel Cryans has posted comments on this change. Change subject: Move the maintenance manager to util .. Patch Set 1: (1 comment) http://gerrit.cloudera.org:8080/#/c/3656/1//COMMIT_MSG Commit Message: PS1, Line 9: it's

[kudu-CR] Integrate the result tracker with writes

2016-07-14 Thread Jean-Daniel Cryans (Code Review)
Jean-Daniel Cryans has posted comments on this change. Change subject: Integrate the result tracker with writes .. Patch Set 25: (14 comments) Just nits. http://gerrit.cloudera.org:8080/#/c/3449/25/src/kudu/tablet/transaction

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

2016-07-14 Thread Jean-Daniel Cryans (Code Review)
Jean-Daniel Cryans 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 16: Code-Review+2 -- To view, visit http://gerrit.cloudera.or

[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 http://ge

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

2016-07-14 Thread Jean-Daniel Cryans (Code Review)
Jean-Daniel Cryans 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: Code-Review+2 (1 comment) http://gerrit.cloudera.org:8080

[kudu-CR] Integrate the request tracker with the client

2016-07-14 Thread Jean-Daniel Cryans (Code Review)
Jean-Daniel Cryans has posted comments on this change. Change subject: Integrate the request tracker with the client .. Patch Set 26: Code-Review+2 -- To view, visit http://gerrit.cloudera.org:8080/3080 To unsubscribe, visit h

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

2016-07-14 Thread Jean-Daniel Cryans (Code Review)
Jean-Daniel Cryans has posted comments on this change. Change subject: Integrate the ResultTracker into the rpc subsystem .. Patch Set 26: Code-Review+2 -- To view, visit http://gerrit.cloudera.org:8080/3192 To unsubscribe, vi

[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 sr

[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: (1 comment) http://gerrit.cloudera.org:8080/#/c/3654/1/java/kudu-client/src/test/java/org/ku

[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) http://gerrit.cloudera.org:8080/#/c/3654/1/java/kudu-client/src/test/java/org/k

[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 T

[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 3: (1 comment) http://gerrit.cloudera.org:8080/#/c/3648/3/src/kudu/client/clie

[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) http://gerrit.cloudera.org:8080/#/c/3648/1/src/kudu/client/cli

[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) http://gerrit.cloudera.org:8080/#/c/3581/4/src/kudu/client/meta_cache.cc

[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 Chan

[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 Ad

[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 http://gerrit

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

2016-07-14 Thread Jean-Daniel Cryans (Code Review)
Jean-Daniel Cryans has posted comments on this change. Change subject: docs: Fix broken link to Introduction in navbar .. Patch Set 1: Code-Review+2 -- To view, visit http://gerrit.cloudera.org:8080/3650 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 4: (1 comment) http://gerrit.cloudera.org:8080/#/c/3614/4/RELEASING.adoc File RELEASING.adoc:

[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 (#8). Change subject: [java client] Redo how we manage exceptions .. [java client] Red

[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 mak

[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 RELEASING.adoc:

[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, Line

[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] I

[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] Red

[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) http://gerrit.cloudera.org:8080/#/c/3055/5/java/kudu-client/src/main/java/org/kudud

[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] Red

[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) http://gerrit.cloudera.org:8080/#/c/3055/5/java/kudu-client/src/main/java/org/kudu

[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] 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 RELEASING.adoc:

[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, Line

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

[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 RELEASING.adoc:

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

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

[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) http://gerrit.cloudera.org:8080/#/c/3631/1/java/kudu-client/src/main/java/org

[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: (5 comments) http://gerrit.cloudera.org:8080/#/c/3631/1/java/kudu-client/src/main/java/or

[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 this

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

2016-07-12 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 (#5). Change subject: [java client] Redo how we manage exceptions .. [java client] Red

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

2016-07-12 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: (4 comments) Thanks a lot Misty for taking my original document to the finish line. http:

[kudu-CR](gh-pages) Add weekly update for 07/11

2016-07-12 Thread Jean-Daniel Cryans (Code Review)
Jean-Daniel Cryans has submitted this change and it was merged. Change subject: Add weekly update for 07/11 .. Add weekly update for 07/11 Change-Id: I5c60067b9e548ff1f75b1cd940646ad6debf462d Reviewed-on: http://gerrit.cloudera

[kudu-CR](gh-pages) Add weekly update for 07/11

2016-07-12 Thread Jean-Daniel Cryans (Code Review)
Jean-Daniel Cryans has posted comments on this change. Change subject: Add weekly update for 07/11 .. Patch Set 5: Verified+1 -- To view, visit http://gerrit.cloudera.org:8080/3616 To unsubscribe, visit http://gerrit.cloudera.

[kudu-CR](gh-pages) Add weekly update for 07/11

2016-07-11 Thread Jean-Daniel Cryans (Code Review)
Hello Mike Percy, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/3616 to look at the new patch set (#5). Change subject: Add weekly update for 07/11 .. Add weekly update for 07/11 Change

[kudu-CR](gh-pages) Add weekly update for 07/11

2016-07-11 Thread Jean-Daniel Cryans (Code Review)
Hello Mike Percy, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/3616 to look at the new patch set (#4). Change subject: Add weekly update for 07/11 .. Add weekly update for 07/11 Change

[kudu-CR] WIP workaround for KUDU-1524

2016-07-11 Thread Jean-Daniel Cryans (Code Review)
Jean-Daniel Cryans has uploaded a new change for review. http://gerrit.cloudera.org:8080/3621 Change subject: WIP workaround for KUDU-1524 .. WIP workaround for KUDU-1524 This is a workaround I used to bring a cluster back up

[kudu-CR](gh-pages) Add weekly update for 07/11

2016-07-11 Thread Jean-Daniel Cryans (Code Review)
Hello Mike Percy, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/3616 to look at the new patch set (#3). Change subject: Add weekly update for 07/11 .. Add weekly update for 07/11 Change

[kudu-CR](gh-pages) Add weekly update for 07/11

2016-07-11 Thread Jean-Daniel Cryans (Code Review)
Jean-Daniel Cryans has posted comments on this change. Change subject: Add weekly update for 07/11 .. Patch Set 2: (3 comments) http://gerrit.cloudera.org:8080/#/c/3616/2/_posts/2016-07-11-weekly-update.md File _posts/2016-07-

[kudu-CR](gh-pages) Add weekly update for 07/11

2016-07-11 Thread Jean-Daniel Cryans (Code Review)
Jean-Daniel Cryans has uploaded a new patch set (#2). Change subject: Add weekly update for 07/11 .. Add weekly update for 07/11 Change-Id: I5c60067b9e548ff1f75b1cd940646ad6debf462d --- A _posts/2016-07-11-weekly-update.md 1 fil

[kudu-CR](gh-pages) Add weekly update for 07/11

2016-07-11 Thread Jean-Daniel Cryans (Code Review)
Jean-Daniel Cryans has posted comments on this change. Change subject: Add weekly update for 07/11 .. Patch Set 1: (2 comments) http://gerrit.cloudera.org:8080/#/c/3616/1/_posts/2016-07-11-weekly-update.md File _posts/2016-07-

[kudu-CR](gh-pages) Add weekly update for 07/11

2016-07-11 Thread Jean-Daniel Cryans (Code Review)
Jean-Daniel Cryans has uploaded a new change for review. http://gerrit.cloudera.org:8080/3616 Change subject: Add weekly update for 07/11 .. Add weekly update for 07/11 Change-Id: I5c60067b9e548ff1f75b1cd940646ad6debf462d ---

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

2016-07-08 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 (#4). Change subject: WIP [java client] Redo how we manage exceptions .. WIP [java cli

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

2016-07-07 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 (#3). Change subject: RFC [java client] Redo how we manage exceptions .. RFC [java cli

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

2016-07-07 Thread Jean-Daniel Cryans (Code Review)
Jean-Daniel Cryans has posted comments on this change. Change subject: RFC [java client] Redo how we manage exceptions .. Patch Set 1: > Build Failed > > http://104.196.14.100/job/kudu-gerrit/2225/ : FAILURE This is a real

[kudu-CR] [java-client] Fix a hang in TabletClient

2016-07-07 Thread Jean-Daniel Cryans (Code Review)
Jean-Daniel Cryans has submitted this change and it was merged. Change subject: [java-client] Fix a hang in TabletClient .. [java-client] Fix a hang in TabletClient TabletClient#sendRpc has been changing a lot lately, and it's

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

2016-07-07 Thread Jean-Daniel Cryans (Code Review)
Jean-Daniel Cryans has posted comments on this change. Change subject: RFC [java client] Redo how we manage exceptions .. Patch Set 2: Latest patch is just a rebase since many things changed in the Java client (although I did

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

2016-07-07 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 (#2). Change subject: RFC [java client] Redo how we manage exceptions .. RFC [java cli

[kudu-CR] [java-client] Fix a hang in TabletClient

2016-07-07 Thread Jean-Daniel Cryans (Code Review)
Jean-Daniel Cryans has posted comments on this change. Change subject: [java-client] Fix a hang in TabletClient .. Patch Set 2: > Looks fine, but the cascading series of fixes to this code has left > me with zero confidence th

[kudu-CR] [java-client] Fix a hang in TabletClient

2016-07-07 Thread Jean-Daniel Cryans (Code Review)
Jean-Daniel Cryans has posted comments on this change. Change subject: [java-client] Fix a hang in TabletClient .. Patch Set 1: (2 comments) http://gerrit.cloudera.org:8080/#/c/3586/1//COMMIT_MSG Commit Message: Line 9: Table

[kudu-CR] [java-client] Fix a hang in TabletClient

2016-07-07 Thread Jean-Daniel Cryans (Code Review)
Hello Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/3586 to look at the new patch set (#2). Change subject: [java-client] Fix a hang in TabletClient .. [java-client] Fix a

  1   2   3   4   >