[kudu-CR] Update Java client for new master GetTableLocations semantics

2016-06-17 Thread Jean-Daniel Cryans (Code Review)
Jean-Daniel Cryans has submitted this change and it was merged. Change subject: Update Java client for new master GetTableLocations semantics .. Update Java client for new master GetTableLocations semantics The Java client was

[kudu-CR] [java-client] use tablet cache for locateTablet calls

2016-06-17 Thread Jean-Daniel Cryans (Code Review)
Jean-Daniel Cryans has submitted this change and it was merged. Change subject: [java-client] use tablet cache for locateTablet calls .. [java-client] use tablet cache for locateTablet calls This simplifies and improves the

[kudu-CR](branch-0.9.x) Bump version to 0.9.1-SNAPSHOT

2016-06-22 Thread Jean-Daniel Cryans (Code Review)
Jean-Daniel Cryans has posted comments on this change. Change subject: Bump version to 0.9.1-SNAPSHOT .. Patch Set 1: Code-Review+2 -- To view, visit http://gerrit.cloudera.org:8080/3458 To unsubscribe, visit

[kudu-CR](branch-0.9.x) Release notes for 0.9.1

2016-06-22 Thread Jean-Daniel Cryans (Code Review)
Jean-Daniel Cryans has posted comments on this change. Change subject: Release notes for 0.9.1 .. Patch Set 1: > Does this belong on the branch or on master? wasn't sure Doh saw this after writing my review. It should be both

[kudu-CR](branch-0.9.x) [java-client] RPC timeout may sometimes be reported as max attempts violation

2016-06-22 Thread Jean-Daniel Cryans (Code Review)
Jean-Daniel Cryans has posted comments on this change. Change subject: [java-client] RPC timeout may sometimes be reported as max attempts violation .. Patch Set 2: Code-Review+2 -- To view, visit

[kudu-CR](gh-pages) Add weekly update for 06/21

2016-06-20 Thread Jean-Daniel Cryans (Code Review)
Jean-Daniel Cryans has uploaded a new change for review. http://gerrit.cloudera.org:8080/3415 Change subject: Add weekly update for 06/21 .. Add weekly update for 06/21 Change-Id: I2d048380922f072e676e990ca23fb2b62da6aa62

[kudu-CR] [java client] Fix GetTableLocations error handling

2016-06-20 Thread Jean-Daniel Cryans (Code Review)
Jean-Daniel Cryans has posted comments on this change. Change subject: [java client] Fix GetTableLocations error handling .. Patch Set 4: Code-Review+2 -- To view, visit http://gerrit.cloudera.org:8080/3295 To unsubscribe,

[kudu-CR] [java client] Fix GetTableLocations error handling

2016-06-20 Thread Jean-Daniel Cryans (Code Review)
Jean-Daniel Cryans has submitted this change and it was merged. Change subject: [java client] Fix GetTableLocations error handling .. [java client] Fix GetTableLocations error handling When GetTableLocations get an error, e.g.

[kudu-CR] [java-client] use tablet cache for locateTablet calls

2016-06-16 Thread Jean-Daniel Cryans (Code Review)
Jean-Daniel Cryans has posted comments on this change. Change subject: [java-client] use tablet cache for locateTablet calls .. Patch Set 2: (2 comments)

[kudu-CR] KUDU-1309: [java client] support tables with non-covering partition-key ranges

2016-06-16 Thread Jean-Daniel Cryans (Code Review)
Jean-Daniel Cryans has posted comments on this change. Change subject: KUDU-1309: [java client] support tables with non-covering partition-key ranges .. Patch Set 2: (3 comments)

[kudu-CR] Update Java client for new master GetTableLocations semantics

2016-06-15 Thread Jean-Daniel Cryans (Code Review)
Jean-Daniel Cryans has posted comments on this change. Change subject: Update Java client for new master GetTableLocations semantics .. Patch Set 4: Code-Review+2 -- To view, visit http://gerrit.cloudera.org:8080/3303 To

[kudu-CR] Release notes for 0.9.1

2016-06-23 Thread Jean-Daniel Cryans (Code Review)
Jean-Daniel Cryans has posted comments on this change. Change subject: Release notes for 0.9.1 .. Patch Set 1: Code-Review+2 -- To view, visit http://gerrit.cloudera.org:8080/3470 To unsubscribe, visit

[kudu-CR](branch-0.9.x) Bump version to 0.9.1

2016-06-23 Thread Jean-Daniel Cryans (Code Review)
Jean-Daniel Cryans has posted comments on this change. Change subject: Bump version to 0.9.1 .. Patch Set 1: Code-Review+2 -- To view, visit http://gerrit.cloudera.org:8080/3473 To unsubscribe, visit

[kudu-CR](gh-pages) Add blog post about multi-master support in Kudu 1.0

2016-06-24 Thread Jean-Daniel Cryans (Code Review)
Jean-Daniel Cryans has posted comments on this change. Change subject: Add blog post about multi-master support in Kudu 1.0 .. Patch Set 1: Verified+1 -- To view, visit http://gerrit.cloudera.org:8080/3476 To unsubscribe,

[kudu-CR](gh-pages) Fix .gitignore

2016-06-24 Thread Jean-Daniel Cryans (Code Review)
Jean-Daniel Cryans has uploaded a new change for review. http://gerrit.cloudera.org:8080/3480 Change subject: Fix .gitignore .. Fix .gitignore Not having untracked files now matters more with 'site_tool publish'. In my case

[kudu-CR](gh-pages) Fix .gitignore

2016-06-24 Thread Jean-Daniel Cryans (Code Review)
Jean-Daniel Cryans has posted comments on this change. Change subject: Fix .gitignore .. Patch Set 1: > this seems kind of fragile.. why not just use a different checkout > for this branch? I'd argue that site_tool is

[kudu-CR](gh-pages) Fix .gitignore

2016-06-24 Thread Jean-Daniel Cryans (Code Review)
Jean-Daniel Cryans has abandoned this change. Change subject: Fix .gitignore .. Abandoned -- To view, visit http://gerrit.cloudera.org:8080/3480 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings

[kudu-CR](gh-pages) Add blog post about removing LocalConsensus

2016-06-16 Thread Jean-Daniel Cryans (Code Review)
Jean-Daniel Cryans has posted comments on this change. Change subject: Add blog post about removing LocalConsensus .. Patch Set 1: (3 comments) I really enjoyed reading this, just minor nits.

[kudu-CR](gh-pages) Add blog post about removing LocalConsensus

2016-06-17 Thread Jean-Daniel Cryans (Code Review)
Jean-Daniel Cryans has posted comments on this change. Change subject: Add blog post about removing LocalConsensus .. Patch Set 3: Code-Review+2 -- To view, visit http://gerrit.cloudera.org:8080/3398 To unsubscribe, visit

[kudu-CR](gh-pages) Add blog post about removing LocalConsensus

2016-06-17 Thread Jean-Daniel Cryans (Code Review)
Jean-Daniel Cryans has submitted this change and it was merged. Change subject: Add blog post about removing LocalConsensus .. Add blog post about removing LocalConsensus Change-Id: I571de0cbf4d73511f17bd57d374fed9e8ca302e4

[kudu-CR] Allow for reserving disk space for non-Kudu processes

2016-06-23 Thread Jean-Daniel Cryans (Code Review)
Jean-Daniel Cryans has posted comments on this change. Change subject: Allow for reserving disk space for non-Kudu processes .. Patch Set 9: Hey Mike, are you planning on doing a follow-up patch with some documentation? Also,

[kudu-CR](branch-0.9.x) Release notes for 0.9.1

2016-06-23 Thread Jean-Daniel Cryans (Code Review)
Jean-Daniel Cryans has posted comments on this change. Change subject: Release notes for 0.9.1 .. Patch Set 3: Code-Review+2 -- To view, visit http://gerrit.cloudera.org:8080/3463 To unsubscribe, visit

[kudu-CR] Add required Debian version to installation page

2016-06-23 Thread Jean-Daniel Cryans (Code Review)
Jean-Daniel Cryans has posted comments on this change. Change subject: Add required Debian version to installation page .. Patch Set 1: > JD, are you ok with pushing this now with your recent deb 8 > testing? Yes. -- To

[kudu-CR](gh-pages) Add blog post about multi-master support in Kudu 1.0

2016-06-23 Thread Jean-Daniel Cryans (Code Review)
Jean-Daniel Cryans has posted comments on this change. Change subject: Add blog post about multi-master support in Kudu 1.0 .. Patch Set 1: Code-Review+2 Good stuff! -- To view, visit http://gerrit.cloudera.org:8080/3476 To

[kudu-CR](gh-pages) Add 6/27 weekly update

2016-06-27 Thread Jean-Daniel Cryans (Code Review)
Jean-Daniel Cryans has posted comments on this change. Change subject: Add 6/27 weekly update .. Patch Set 1: (1 comment) http://gerrit.cloudera.org:8080/#/c/3515/1/_posts/2016-06-27-weekly-update.md File

[kudu-CR] KUDU-1496. NoLeaderMasterFoundException are mishandled

2016-06-27 Thread Jean-Daniel Cryans (Code Review)
Jean-Daniel Cryans has posted comments on this change. Change subject: KUDU-1496. NoLeaderMasterFoundException are mishandled .. Patch Set 1: (1 comment) > Oops. > > Outside of multi-master, this only manifests when the

[kudu-CR] KUDU-1496. NoLeaderMasterFoundException are mishandled

2016-06-27 Thread Jean-Daniel Cryans (Code Review)
Jean-Daniel Cryans has submitted this change and it was merged. Change subject: KUDU-1496. NoLeaderMasterFoundException are mishandled .. KUDU-1496. NoLeaderMasterFoundException are mishandled This is a regression from

[kudu-CR] KUDU-1496. NoLeaderMasterFoundException are mishandled

2016-06-27 Thread Jean-Daniel Cryans (Code Review)
Jean-Daniel Cryans has posted comments on this change. Change subject: KUDU-1496. NoLeaderMasterFoundException are mishandled .. Patch Set 1: Verified+1 Forcing +1 due to current messed up workspace. -- To view, visit

[kudu-CR] tablet: change default bloom filter FP rate to 0.01%

2016-06-27 Thread Jean-Daniel Cryans (Code Review)
Jean-Daniel Cryans has posted comments on this change. Change subject: tablet: change default bloom filter FP rate to 0.01% .. Patch Set 1: Code-Review+2 -- To view, visit http://gerrit.cloudera.org:8080/3517 To unsubscribe,

[kudu-CR](gh-pages) Add 6/27 weekly update

2016-06-27 Thread Jean-Daniel Cryans (Code Review)
Jean-Daniel Cryans has posted comments on this change. Change subject: Add 6/27 weekly update .. Patch Set 2: Code-Review+2 -- To view, visit http://gerrit.cloudera.org:8080/3515 To unsubscribe, visit

[kudu-CR] KUDU-1496. NoLeaderMasterFoundException are mishandled

2016-06-27 Thread Jean-Daniel Cryans (Code Review)
Jean-Daniel Cryans has uploaded a new change for review. http://gerrit.cloudera.org:8080/3516 Change subject: KUDU-1496. NoLeaderMasterFoundException are mishandled .. KUDU-1496. NoLeaderMasterFoundException are mishandled

[kudu-CR] tablet: change default bloom filter FP rate to 0.01%

2016-06-27 Thread Jean-Daniel Cryans (Code Review)
Jean-Daniel Cryans has posted comments on this change. Change subject: tablet: change default bloom filter FP rate to 0.01% .. Patch Set 2: Code-Review+2 (1 comment) http://gerrit.cloudera.org:8080/#/c/3517/2//COMMIT_MSG

[kudu-CR] [java-client] refactor AsyncKuduSession

2016-06-24 Thread Jean-Daniel Cryans (Code Review)
Jean-Daniel Cryans has posted comments on this change. Change subject: [java-client] refactor AsyncKuduSession .. Patch Set 1: (7 comments)

[kudu-CR] [java-client] refactor AsyncKuduSession

2016-06-24 Thread Jean-Daniel Cryans (Code Review)
Jean-Daniel Cryans has posted comments on this change. Change subject: [java-client] refactor AsyncKuduSession .. Patch Set 2: (1 comment)

[kudu-CR] Reduce verbosity of Java logs

2016-05-24 Thread Jean-Daniel Cryans (Code Review)
Jean-Daniel Cryans has posted comments on this change. Change subject: Reduce verbosity of Java logs .. Patch Set 1: (1 comment) http://gerrit.cloudera.org:8080/#/c/3203/1/java/kudu-client/src/test/resources/log4j.properties

[kudu-CR] Reduce verbosity of Java logs

2016-05-24 Thread Jean-Daniel Cryans (Code Review)
Jean-Daniel Cryans has posted comments on this change. Change subject: Reduce verbosity of Java logs .. Patch Set 1: (1 comment) http://gerrit.cloudera.org:8080/#/c/3203/1/java/kudu-client/src/test/resources/log4j.properties

[kudu-CR] service_pool: only log queue overflows once per second

2016-05-24 Thread Jean-Daniel Cryans (Code Review)
Jean-Daniel Cryans has posted comments on this change. Change subject: service_pool: only log queue overflows once per second .. Patch Set 1: Code-Review+2 Looks good for branch-0.9.x too. -- To view, visit

[kudu-CR] Bump the master branch's version to 1.0.0-SNAPSHOT

2016-05-24 Thread Jean-Daniel Cryans (Code Review)
Jean-Daniel Cryans has submitted this change and it was merged. Change subject: Bump the master branch's version to 1.0.0-SNAPSHOT .. Bump the master branch's version to 1.0.0-SNAPSHOT Change-Id:

[kudu-CR](branch-0.9.x) rpcz: break out trace metrics into separate PB fields

2016-05-24 Thread Jean-Daniel Cryans (Code Review)
Jean-Daniel Cryans has posted comments on this change. Change subject: rpcz: break out trace metrics into separate PB fields .. Patch Set 1: Code-Review+2 -- To view, visit http://gerrit.cloudera.org:8080/3195 To unsubscribe,

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

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

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

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

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

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

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

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

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

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

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

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

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

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

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

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

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

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

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

[kudu-CR] Memory tracking for result tracker

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

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

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

[kudu-CR] 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

[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

[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

[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

[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

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

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

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

[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

[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

[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

[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

[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

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

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

[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

[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

[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)
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] [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: (6 comments)

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

[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

[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

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

<    1   2   3   >