[kudu-CR] maintenance manager-test: fix flaky test

2016-10-05 Thread Jean-Daniel Cryans (Code Review)
Jean-Daniel Cryans has posted comments on this change. Change subject: maintenance_manager-test: fix flaky test .. Patch Set 1: Code-Review+2 -- To view, visit http://gerrit.cloudera.org:8080/4634 To unsubscribe, visit

[kudu-CR] [java client] Temporarily ignore row count regressions in ITClient

2016-10-05 Thread Jean-Daniel Cryans (Code Review)
Jean-Daniel Cryans has posted comments on this change. Change subject: [java client] Temporarily ignore row count regressions in ITClient .. Patch Set 2: (1 comment)

[kudu-CR] [java client] Temporarily ignore row count regressions in ITClient

2016-10-05 Thread Jean-Daniel Cryans (Code Review)
Hello David Ribeiro Alves, Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/4597 to look at the new patch set (#2). Change subject: [java client] Temporarily ignore row count regressions in ITClient

[kudu-CR] [java client] Rename NoLeaderMasterFoundException and reuse

2016-10-05 Thread Jean-Daniel Cryans (Code Review)
Hello Dan Burkert, Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/4624 to look at the new patch set (#2). Change subject: [java client] Rename NoLeaderMasterFoundException and reuse

[kudu-CR] [java client] Rename NoLeaderMasterFoundException and reuse

2016-10-04 Thread Jean-Daniel Cryans (Code Review)
Jean-Daniel Cryans has uploaded a new change for review. http://gerrit.cloudera.org:8080/4624 Change subject: [java client] Rename NoLeaderMasterFoundException and reuse .. [java client] Rename NoLeaderMasterFoundException

[kudu-CR] [java client] Tight-ish loop in master lookups if a tablet doesn't have a leader

2016-10-04 Thread Jean-Daniel Cryans (Code Review)
Jean-Daniel Cryans has posted comments on this change. Change subject: [java client] Tight-ish loop in master lookups if a tablet doesn't have a leader .. Patch Set 3: Verified+1 Dan messed up my build, but I did run all the

[kudu-CR] [java client] Tight-ish loop in master lookups if a tablet doesn't have a leader

2016-10-04 Thread Jean-Daniel Cryans (Code Review)
Jean-Daniel Cryans has submitted this change and it was merged. Change subject: [java client] Tight-ish loop in master lookups if a tablet doesn't have a leader .. [java client] Tight-ish loop in master lookups if a tablet

[kudu-CR] [java client] Tight-ish loop in master lookups if a tablet doesn't have a leader

2016-10-04 Thread Jean-Daniel Cryans (Code Review)
Hello Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/4570 to look at the new patch set (#3). Change subject: [java client] Tight-ish loop in master lookups if a tablet doesn't have a leader

[kudu-CR] [java client] Tight-ish loop in master lookups if a tablet doesn't have a leader

2016-10-04 Thread Jean-Daniel Cryans (Code Review)
Jean-Daniel Cryans has posted comments on this change. Change subject: [java client] Tight-ish loop in master lookups if a tablet doesn't have a leader .. Patch Set 2: (2 comments)

[kudu-CR] KUDU-1652 (part 2): Filter IS NOT NULL predicates from scan on client

2016-10-04 Thread Jean-Daniel Cryans (Code Review)
Jean-Daniel Cryans has posted comments on this change. Change subject: KUDU-1652 (part 2): Filter IS NOT NULL predicates from scan on client .. Patch Set 1: Code-Review+2 -- To view, visit

[kudu-CR] [java client] Tight-ish loop in master lookups if a tablet doesn't have a leader

2016-10-04 Thread Jean-Daniel Cryans (Code Review)
Jean-Daniel Cryans has posted comments on this change. Change subject: [java client] Tight-ish loop in master lookups if a tablet doesn't have a leader .. Patch Set 2: (2 comments)

[kudu-CR] Separate tablet MM ops into separate .cc file

2016-10-04 Thread Jean-Daniel Cryans (Code Review)
Jean-Daniel Cryans has posted comments on this change. Change subject: Separate tablet MM ops into separate .cc file .. Patch Set 6: Code-Review+2 I didn't run a line-by-line comparison of your copy paste so I assume it's fine

[kudu-CR] java: fix kudu-client builds with TSAN-only thirdparty

2016-10-03 Thread Jean-Daniel Cryans (Code Review)
Jean-Daniel Cryans has posted comments on this change. Change subject: java: fix kudu-client builds with TSAN-only thirdparty .. Patch Set 1: Code-Review+2 -- To view, visit http://gerrit.cloudera.org:8080/4577 To

[kudu-CR](branch-1.0.x) [java client] make DateFormat safe to use

2016-10-03 Thread Jean-Daniel Cryans (Code Review)
Jean-Daniel Cryans has submitted this change and it was merged. Change subject: [java client] make DateFormat safe to use .. [java client] make DateFormat safe to use Todd's error-prone patch discovered this problem in

[kudu-CR](branch-1.0.x) [java client] make DateFormat safe to use

2016-10-03 Thread Jean-Daniel Cryans (Code Review)
Jean-Daniel Cryans has posted comments on this change. Change subject: [java client] make DateFormat safe to use .. Patch Set 1: Code-Review+2 Verified+1 Unrelated C++ test failure. -- To view, visit

[kudu-CR](branch-1.0.x) [java client] Fix an NPE in KuduException

2016-10-03 Thread Jean-Daniel Cryans (Code Review)
Jean-Daniel Cryans has submitted this change and it was merged. Change subject: [java client] Fix an NPE in KuduException .. [java client] Fix an NPE in KuduException Saw this in a Jenkins run and also running ITClient on my

[kudu-CR](branch-1.0.x) [java client] Fix an NPE in KuduException

2016-10-03 Thread Jean-Daniel Cryans (Code Review)
Jean-Daniel Cryans has posted comments on this change. Change subject: [java client] Fix an NPE in KuduException .. Patch Set 1: Code-Review+2 -- To view, visit http://gerrit.cloudera.org:8080/4605 To unsubscribe, visit

[kudu-CR] KUDU-1669. Java ITClient test can orphan processes

2016-10-03 Thread Jean-Daniel Cryans (Code Review)
Jean-Daniel Cryans has uploaded a new change for review. http://gerrit.cloudera.org:8080/4598 Change subject: KUDU-1669. Java ITClient test can orphan processes .. KUDU-1669. Java ITClient test can orphan processes

[kudu-CR] [java client] Temporarily ignore row count regressions in ITClient

2016-10-03 Thread Jean-Daniel Cryans (Code Review)
Jean-Daniel Cryans has uploaded a new change for review. http://gerrit.cloudera.org:8080/4597 Change subject: [java client] Temporarily ignore row count regressions in ITClient .. [java client] Temporarily ignore row count

[kudu-CR] [java client] Tight-ish loop in master lookups if a tablet doesn't have a leader

2016-10-03 Thread Jean-Daniel Cryans (Code Review)
Jean-Daniel Cryans has posted comments on this change. Change subject: [java client] Tight-ish loop in master lookups if a tablet doesn't have a leader .. Patch Set 1: (7 comments)

[kudu-CR] [java client] Tight-ish loop in master lookups if a tablet doesn't have a leader

2016-09-29 Thread Jean-Daniel Cryans (Code Review)
Jean-Daniel Cryans has uploaded a new change for review. http://gerrit.cloudera.org:8080/4570 Change subject: [java client] Tight-ish loop in master lookups if a tablet doesn't have a leader .. [java client] Tight-ish loop

[kudu-CR] [docs] Cleanup beta mentions, links

2016-09-29 Thread Jean-Daniel Cryans (Code Review)
Jean-Daniel Cryans has submitted this change and it was merged. Change subject: [docs] Cleanup beta mentions, links .. [docs] Cleanup beta mentions, links Change-Id: I56fa65be9d027f5b40628aa5db5f9e4a57c1fbc9 Reviewed-on:

[kudu-CR] [docs] Release note for OperationResponse.getWriteTimestamp

2016-09-29 Thread Jean-Daniel Cryans (Code Review)
Jean-Daniel Cryans has posted comments on this change. Change subject: [docs] Release note for OperationResponse.getWriteTimestamp .. Patch Set 5: Verified+1 -- To view, visit http://gerrit.cloudera.org:8080/4560 To

[kudu-CR] [docs] Release note for OperationResponse.getWriteTimestamp

2016-09-29 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/4560 to look at the new patch set (#5). Change subject: [docs] Release note for OperationResponse.getWriteTimestamp

[kudu-CR] [docs] Release note for OperationResponse.getWriteTimestamp

2016-09-28 Thread Jean-Daniel Cryans (Code Review)
Jean-Daniel Cryans has posted comments on this change. Change subject: [docs] Release note for OperationResponse.getWriteTimestamp .. Patch Set 1: (1 comment) http://gerrit.cloudera.org:8080/#/c/4560/1/docs/release_notes.adoc

[kudu-CR] [docs] Release note for OperationResponse.getWriteTimestamp

2016-09-28 Thread Jean-Daniel Cryans (Code Review)
Hello Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/4560 to look at the new patch set (#4). Change subject: [docs] Release note for OperationResponse.getWriteTimestamp ..

[kudu-CR] [docs] Release note for OperationResponse.getWriteTimestamp

2016-09-28 Thread Jean-Daniel Cryans (Code Review)
Jean-Daniel Cryans has posted comments on this change. Change subject: [docs] Release note for OperationResponse.getWriteTimestamp .. Patch Set 1: (2 comments)

[kudu-CR] [docs] Release note for OperationResponse.getWriteTimestamp

2016-09-28 Thread Jean-Daniel Cryans (Code Review)
Hello Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/4560 to look at the new patch set (#3). Change subject: [docs] Release note for OperationResponse.getWriteTimestamp ..

[kudu-CR] [docs] Release note for OperationResponse.getWriteTimestamp

2016-09-28 Thread Jean-Daniel Cryans (Code Review)
Jean-Daniel Cryans has uploaded a new change for review. http://gerrit.cloudera.org:8080/4560 Change subject: [docs] Release note for OperationResponse.getWriteTimestamp .. [docs] Release note for

[kudu-CR] [java-client] IN-list predicate

2016-09-28 Thread Jean-Daniel Cryans (Code Review)
Jean-Daniel Cryans has posted comments on this change. Change subject: [java-client] IN-list predicate .. Patch Set 6: Code-Review+2 -- To view, visit http://gerrit.cloudera.org:8080/4530 To unsubscribe, visit

[kudu-CR] [java client] ITClient's new snapshot scanners can easily timeout

2016-09-27 Thread Jean-Daniel Cryans (Code Review)
Jean-Daniel Cryans has submitted this change and it was merged. Change subject: [java client] ITClient's new snapshot scanners can easily timeout .. [java client] ITClient's new snapshot scanners can easily timeout This patch

[kudu-CR] [java client] ITClient's new snapshot scanners can easily timeout

2016-09-27 Thread Jean-Daniel Cryans (Code Review)
Hello Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/4545 to look at the new patch set (#2). Change subject: [java client] ITClient's new snapshot scanners can easily timeout

[kudu-CR] [java client] ITClient's new snapshot scanners can easily timeout

2016-09-27 Thread Jean-Daniel Cryans (Code Review)
Jean-Daniel Cryans has posted comments on this change. Change subject: [java client] ITClient's new snapshot scanners can easily timeout .. Patch Set 1: (1 comment)

[kudu-CR] [java client] ITClient's new snapshot scanners can easily timeout

2016-09-27 Thread Jean-Daniel Cryans (Code Review)
Jean-Daniel Cryans has posted comments on this change. Change subject: [java client] ITClient's new snapshot scanners can easily timeout .. Patch Set 1: (1 comment)

[kudu-CR] [java client] ITClient's new snapshot scanners can easily timeout

2016-09-27 Thread Jean-Daniel Cryans (Code Review)
Jean-Daniel Cryans has uploaded a new change for review. http://gerrit.cloudera.org:8080/4545 Change subject: [java client] ITClient's new snapshot scanners can easily timeout .. [java client] ITClient's new snapshot

[kudu-CR] [java-client] IN-list predicate

2016-09-27 Thread Jean-Daniel Cryans (Code Review)
Jean-Daniel Cryans has posted comments on this change. Change subject: [java-client] IN-list predicate .. Patch Set 4: Code-Review+2 -- To view, visit http://gerrit.cloudera.org:8080/4530 To unsubscribe, visit

[kudu-CR] [java-client] IN-list predicate

2016-09-27 Thread Jean-Daniel Cryans (Code Review)
Jean-Daniel Cryans has posted comments on this change. Change subject: [java-client] IN-list predicate .. Patch Set 3: (1 comment)

[kudu-CR] [java] KUDU-1563. Add support for INSERT IGNORE

2016-09-27 Thread Jean-Daniel Cryans (Code Review)
Jean-Daniel Cryans has posted comments on this change. Change subject: [java] KUDU-1563. Add support for INSERT IGNORE .. Patch Set 7: (6 comments)

[kudu-CR] KUDU-1563. Add support for INSERT IGNORE

2016-09-27 Thread Jean-Daniel Cryans (Code Review)
Jean-Daniel Cryans has posted comments on this change. Change subject: KUDU-1563. Add support for INSERT IGNORE .. Patch Set 7: (1 comment) http://gerrit.cloudera.org:8080/#/c/4491/7/src/kudu/tablet/tablet-test.cc File

[kudu-CR] KUDU-1652: Partition pruning fails with IS NOT NULL predicate on PK column

2016-09-27 Thread Jean-Daniel Cryans (Code Review)
Jean-Daniel Cryans has posted comments on this change. Change subject: KUDU-1652: Partition pruning fails with IS NOT NULL predicate on PK column .. Patch Set 1: Code-Review+2 -- To view, visit

[kudu-CR](gh-pages) Add post about Strata talks

2016-09-26 Thread Jean-Daniel Cryans (Code Review)
Jean-Daniel Cryans has posted comments on this change. Change subject: Add post about Strata talks .. Patch Set 2: Code-Review+2 -- To view, visit http://gerrit.cloudera.org:8080/4538 To unsubscribe, visit

[kudu-CR](gh-pages) Add post about Strata talks

2016-09-26 Thread Jean-Daniel Cryans (Code Review)
Jean-Daniel Cryans has posted comments on this change. Change subject: Add post about Strata talks .. Patch Set 1: (2 comments) http://gerrit.cloudera.org:8080/#/c/4538/1/_posts/2016-09-26-strata-nyc-kudu-talks.md File

[kudu-CR] [java-client] IN-list predicate

2016-09-26 Thread Jean-Daniel Cryans (Code Review)
Jean-Daniel Cryans has posted comments on this change. Change subject: [java-client] IN-list predicate .. Patch Set 1: (2 comments)

[kudu-CR] release notes: prepare for 1.1

2016-09-26 Thread Jean-Daniel Cryans (Code Review)
Jean-Daniel Cryans has posted comments on this change. Change subject: release_notes: prepare for 1.1 .. Patch Set 1: Code-Review+2 -- To view, visit http://gerrit.cloudera.org:8080/4532 To unsubscribe, visit

[kudu-CR] [WIP] KUDU-1563. Add support for INSERT IGNORE

2016-09-22 Thread Jean-Daniel Cryans (Code Review)
Jean-Daniel Cryans has posted comments on this change. Change subject: [WIP] KUDU-1563. Add support for INSERT IGNORE .. Patch Set 2: (1 comment)

[kudu-CR] Document Impala and Spark integration known issues & limitations

2016-09-22 Thread Jean-Daniel Cryans (Code Review)
Jean-Daniel Cryans has posted comments on this change. Change subject: Document Impala and Spark integration known issues & limitations .. Patch Set 3: (1 comment)

[kudu-CR] [website] broken link to an adoc

2016-09-21 Thread Jean-Daniel Cryans (Code Review)
Jean-Daniel Cryans has uploaded a new change for review. http://gerrit.cloudera.org:8080/4504 Change subject: [website] broken link to an adoc .. [website] broken link to an adoc Change-Id:

[kudu-CR] [java client] Reinstate KUDU-1364's behavior, fix NPE

2016-09-21 Thread Jean-Daniel Cryans (Code Review)
Jean-Daniel Cryans has uploaded a new change for review. http://gerrit.cloudera.org:8080/4501 Change subject: [java client] Reinstate KUDU-1364's behavior, fix NPE .. [java client] Reinstate KUDU-1364's behavior, fix NPE

[kudu-CR] [java client] Few ITClient improvements

2016-09-21 Thread Jean-Daniel Cryans (Code Review)
Jean-Daniel Cryans has submitted this change and it was merged. Change subject: [java client] Few ITClient improvements .. [java client] Few ITClient improvements ITClient has been flaky for a while now, mostly due to the "Row

[kudu-CR] [java client] Fix an NPE in KuduException

2016-09-21 Thread Jean-Daniel Cryans (Code Review)
Jean-Daniel Cryans has submitted this change and it was merged. Change subject: [java client] Fix an NPE in KuduException .. [java client] Fix an NPE in KuduException Saw this in a Jenkins run and also running ITClient on my

[kudu-CR] [java client] Improve and hide OperationResponse#getWriteTimestamp

2016-09-21 Thread Jean-Daniel Cryans (Code Review)
Jean-Daniel Cryans has submitted this change and it was merged. Change subject: [java client] Improve and hide OperationResponse#getWriteTimestamp .. [java client] Improve and hide OperationResponse#getWriteTimestamp That

[kudu-CR] [java client] Improve and hide OperationResponse#getWriteTimestamp

2016-09-21 Thread Jean-Daniel Cryans (Code Review)
Jean-Daniel Cryans has posted comments on this change. Change subject: [java client] Improve and hide OperationResponse#getWriteTimestamp .. Patch Set 1: (1 comment)

[kudu-CR] [java client] Few ITClient improvements

2016-09-21 Thread Jean-Daniel Cryans (Code Review)
Hello Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/4489 to look at the new patch set (#3). Change subject: [java client] Few ITClient improvements .. [java client] Few

[kudu-CR] [java client] Few ITClient improvements

2016-09-21 Thread Jean-Daniel Cryans (Code Review)
Jean-Daniel Cryans has posted comments on this change. Change subject: [java client] Few ITClient improvements .. Patch Set 1: (1 comment)

[kudu-CR] [java client] Few ITClient improvements

2016-09-21 Thread Jean-Daniel Cryans (Code Review)
Hello Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/4489 to look at the new patch set (#2). Change subject: [java client] Few ITClient improvements .. [java client] Few

[kudu-CR] [java client] Few ITClient improvements

2016-09-21 Thread Jean-Daniel Cryans (Code Review)
Jean-Daniel Cryans has posted comments on this change. Change subject: [java client] Few ITClient improvements .. Patch Set 1: (6 comments)

[kudu-CR] [java client] Improve and hide OperationResponse#getWriteTimestamp

2016-09-21 Thread Jean-Daniel Cryans (Code Review)
Jean-Daniel Cryans has posted comments on this change. Change subject: [java client] Improve and hide OperationResponse#getWriteTimestamp .. Patch Set 1: (1 comment)

[kudu-CR] [java client] Fix an NPE in KuduException

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

[kudu-CR] [java client] Improve and hide OperationResponse#getWriteTimestamp

2016-09-21 Thread Jean-Daniel Cryans (Code Review)
Hello Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/4487 to look at the new patch set (#2). Change subject: [java client] Improve and hide OperationResponse#getWriteTimestamp

[kudu-CR] [java client] Improve and hide OperationResponse#getWriteTimestamp

2016-09-21 Thread Jean-Daniel Cryans (Code Review)
Jean-Daniel Cryans has posted comments on this change. Change subject: [java client] Improve and hide OperationResponse#getWriteTimestamp .. Patch Set 1: (2 comments)

[kudu-CR] [java client] Fix an NPE in KuduException

2016-09-20 Thread Jean-Daniel Cryans (Code Review)
Jean-Daniel Cryans has posted comments on this change. Change subject: [java client] Fix an NPE in KuduException .. Patch Set 1: (1 comment)

[kudu-CR] [java client] Improve and hide OperationResponse#getWriteTimestamp

2016-09-20 Thread Jean-Daniel Cryans (Code Review)
Jean-Daniel Cryans has uploaded a new change for review. http://gerrit.cloudera.org:8080/4487 Change subject: [java client] Improve and hide OperationResponse#getWriteTimestamp .. [java client] Improve and hide

[kudu-CR] [java client] Fix an NPE in KuduException

2016-09-20 Thread Jean-Daniel Cryans (Code Review)
Jean-Daniel Cryans has uploaded a new change for review. http://gerrit.cloudera.org:8080/4488 Change subject: [java client] Fix an NPE in KuduException .. [java client] Fix an NPE in KuduException Saw this in a Jenkins run

[kudu-CR] [java client] Few ITClient improvements

2016-09-20 Thread Jean-Daniel Cryans (Code Review)
Jean-Daniel Cryans has uploaded a new change for review. http://gerrit.cloudera.org:8080/4489 Change subject: [java client] Few ITClient improvements .. [java client] Few ITClient improvements ITClient has been flaky for a

[kudu-CR] c++ client: use entire deadline when retrying single-master DDL operations

2016-09-20 Thread Jean-Daniel Cryans (Code Review)
Jean-Daniel Cryans has submitted this change and it was merged. Change subject: c++ client: use entire deadline when retrying single-master DDL operations .. c++ client: use entire deadline when retrying single-master DDL

[kudu-CR] [java-client] address follow-up comments to c7dab4

2016-09-16 Thread Jean-Daniel Cryans (Code Review)
Jean-Daniel Cryans has posted comments on this change. Change subject: [java-client] address follow-up comments to c7dab4 .. Patch Set 1: Code-Review+2 -- To view, visit http://gerrit.cloudera.org:8080/4442 To unsubscribe,

[kudu-CR] [java client] KuduScanToken#build cleanup

2016-09-15 Thread Jean-Daniel Cryans (Code Review)
Jean-Daniel Cryans has uploaded a new change for review. http://gerrit.cloudera.org:8080/4430 Change subject: [java client] KuduScanToken#build cleanup .. [java client] KuduScanToken#build cleanup Using Todd's patch for

[kudu-CR] Set desiredBlockSize in pbToColumnSchema

2016-09-14 Thread Jean-Daniel Cryans (Code Review)
Jean-Daniel Cryans has posted comments on this change. Change subject: Set desiredBlockSize in pbToColumnSchema .. Patch Set 2: Code-Review+2 (1 comment) http://gerrit.cloudera.org:8080/#/c/4407/2//COMMIT_MSG Commit Message:

[kudu-CR] Set desiredBlockSize in pbToColumnSchema

2016-09-14 Thread Jean-Daniel Cryans (Code Review)
Jean-Daniel Cryans has submitted this change and it was merged. Change subject: Set desiredBlockSize in pbToColumnSchema .. Set desiredBlockSize in pbToColumnSchema pbToColumnSchema forgets to set desiredBlockSize; this fixes

[kudu-CR] [java] - Synchronize the removal of the TabletClient from client2tablets and ip2client

2016-09-13 Thread Jean-Daniel Cryans (Code Review)
Jean-Daniel Cryans has submitted this change and it was merged. Change subject: [java] - Synchronize the removal of the TabletClient from client2tablets and ip2client .. [java] - Synchronize the removal of the TabletClient

[kudu-CR] [java] - Synchronize the removal of the TabletClient from client2tablets and ip2client

2016-09-13 Thread Jean-Daniel Cryans (Code Review)
Jean-Daniel Cryans has posted comments on this change. Change subject: [java] - Synchronize the removal of the TabletClient from client2tablets and ip2client .. Patch Set 2: Code-Review+2 -- To view, visit

[kudu-CR] Allow multi-master AlterTable tests in TestKuduTable.java

2016-09-13 Thread Jean-Daniel Cryans (Code Review)
Jean-Daniel Cryans has posted comments on this change. Change subject: Allow multi-master AlterTable tests in TestKuduTable.java .. Patch Set 3: Code-Review+2 -- To view, visit http://gerrit.cloudera.org:8080/4406 To

[kudu-CR] Allow multi-master AlterTable tests in TestKuduTable.java

2016-09-13 Thread Jean-Daniel Cryans (Code Review)
Jean-Daniel Cryans has posted comments on this change. Change subject: Allow multi-master AlterTable tests in TestKuduTable.java .. Patch Set 2: Code-Review+2 Assuming the test passes. -- To view, visit

[kudu-CR] Allow multi-master AlterTable tests in TestKuduTable.java

2016-09-13 Thread Jean-Daniel Cryans (Code Review)
Jean-Daniel Cryans has posted comments on this change. Change subject: Allow multi-master AlterTable tests in TestKuduTable.java .. Patch Set 1: (1 comment) http://gerrit.cloudera.org:8080/#/c/4406/1//COMMIT_MSG Commit

[kudu-CR](gh-pages) Blogpost describing predicate evaluation pushdown

2016-09-12 Thread Jean-Daniel Cryans (Code Review)
Jean-Daniel Cryans has posted comments on this change. Change subject: Blogpost describing predicate evaluation pushdown .. Patch Set 2: (2 comments) http://gerrit.cloudera.org:8080/#/c/4384/2/_posts/predicate-pushdown.md

[kudu-CR] [java] - Synchronize the removal of the TabletClient from client2tablets and ip2client

2016-09-12 Thread Jean-Daniel Cryans (Code Review)
Jean-Daniel Cryans has posted comments on this change. Change subject: [java] - Synchronize the removal of the TabletClient from client2tablets and ip2client .. Patch Set 2: (1 comment) How did you hit this originally?

[kudu-CR] KUDU-1542: in some cases apply will hang.

2016-07-26 Thread Jean-Daniel Cryans (Code Review)
Jean-Daniel Cryans has posted comments on this change. Change subject: KUDU-1542: in some cases apply will hang. .. Patch Set 2: (1 comment)

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

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

[kudu-CR](gh-pages) www: Redirect kudu.i.a.o to kudu.a.o

2016-07-25 Thread Jean-Daniel Cryans (Code Review)
Jean-Daniel Cryans has posted comments on this change. Change subject: www: Redirect kudu.i.a.o to kudu.a.o .. Patch Set 1: Been years since I wrote htaccess stuff, so I'll trust you on that one :) -- To view, visit

[kudu-CR](gh-pages) Blog post for the graduation

2016-07-25 Thread Jean-Daniel Cryans (Code Review)
Jean-Daniel Cryans has uploaded a new patch set (#2). Change subject: Blog post for the graduation .. Blog post for the graduation This is just a reblog with the appropriate markdown syntax. Change-Id:

[kudu-CR] [java-client] repackage to org.apache.kudu (Part 2)

2016-07-25 Thread Jean-Daniel Cryans (Code Review)
Jean-Daniel Cryans has posted comments on this change. Change subject: [java-client] repackage to org.apache.kudu (Part 2) .. Patch Set 1: Code-Review+2 -- To view, visit http://gerrit.cloudera.org:8080/3737 To unsubscribe,

[kudu-CR] [java-client] repackage to org.apache.kudu (Part 1)

2016-07-25 Thread Jean-Daniel Cryans (Code Review)
Jean-Daniel Cryans has submitted this change and it was merged. Change subject: [java-client] repackage to org.apache.kudu (Part 1) .. [java-client] repackage to org.apache.kudu (Part 1) Move all org.kududb directories to

[kudu-CR] ksck: improve filtering capability

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

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

2016-07-22 Thread Jean-Daniel Cryans (Code Review)
Jean-Daniel Cryans has submitted this change and it was merged. Change subject: ksck: also print info about the good replicas for a bad tablet .. ksck: also print info about the good replicas for a bad tablet In the case that

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

2016-07-22 Thread Jean-Daniel Cryans (Code Review)
Jean-Daniel Cryans has submitted this change and it was merged. Change subject: KUDU-1516 ksck should check for more raft-related status issues (partial) .. KUDU-1516 ksck should check for more raft-related status issues

<    3   4   5   6   7   8