[kudu-CR] ts itest-base.h: wait for bootstrapping to finish when waiting for replicas

2016-06-07 Thread Todd Lipcon (Code Review)
Todd Lipcon has posted comments on this change. Change subject: ts_itest-base.h: wait for bootstrapping to finish when waiting for replicas .. Patch Set 2: Code-Review+2 -- To view, visit http://gerrit.cloudera.org:8080/3308

[kudu-CR] KUDU-1469. Fix handling of fully-deduped requests after a leader change

2016-06-07 Thread Todd Lipcon (Code Review)
Todd Lipcon has posted comments on this change. Change subject: KUDU-1469. Fix handling of fully-deduped requests after a leader change .. Patch Set 3: (5 comments) http://gerrit.cloudera.org:8080/#/c/3228/3/src/kudu/consensu

[kudu-CR] KUDU-1469. Fix handling of fully-deduped requests after a leader change

2016-06-07 Thread Todd Lipcon (Code Review)
Hello Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/3228 to look at the new patch set (#4). Change subject: KUDU-1469. Fix handling of fully-deduped requests after a leader change

[kudu-CR] KUDU-1469. Fix handling of fully-deduped requests after a leader change

2016-06-07 Thread Kudu Jenkins (Code Review)
Kudu Jenkins has posted comments on this change. Change subject: KUDU-1469. Fix handling of fully-deduped requests after a leader change .. Patch Set 4: Build Started http://104.196.14.100/job/kudu-gerrit/1756/ -- To view, v

[kudu-CR] KUDU-1473: fix some tablet lock usage in CatalogManager

2016-06-07 Thread Todd Lipcon (Code Review)
Todd Lipcon has posted comments on this change. Change subject: KUDU-1473: fix some tablet lock usage in CatalogManager .. Patch Set 2: (1 comment) http://gerrit.cloudera.org:8080/#/c/3309/2//COMMIT_MSG Commit Message: Line 1

[kudu-CR] KUDU-1469. Fix handling of fully-deduped requests after a leader change

2016-06-07 Thread Kudu Jenkins (Code Review)
Kudu Jenkins has posted comments on this change. Change subject: KUDU-1469. Fix handling of fully-deduped requests after a leader change .. Patch Set 4: -Verified Build Started http://104.196.14.100/job/kudu-gerrit/1757/ --

[kudu-CR] KUDU-1353: remove per-tablet replica locations cache

2016-06-07 Thread Todd Lipcon (Code Review)
Todd Lipcon has posted comments on this change. Change subject: KUDU-1353: remove per-tablet replica locations cache .. Patch Set 8: (2 comments) http://gerrit.cloudera.org:8080/#/c/2887/8/src/kudu/master/catalog_manager.cc Fi

[kudu-CR] KUDU-1398 CFile index blocks can store shortest separating prefix

2016-06-07 Thread Todd Lipcon (Code Review)
Todd Lipcon has posted comments on this change. Change subject: KUDU-1398 CFile index blocks can store shortest separating prefix .. Patch Set 4: (15 comments) http://gerrit.cloudera.org:8080/#/c/3304/4//COMMIT_MSG Commit Mes

[kudu-CR] WIP: KUDU-1466: improve error message when writes fail at TS

2016-06-07 Thread Todd Lipcon (Code Review)
Hello Adar Dembo, I'd like you to do a code review. Please visit http://gerrit.cloudera.org:8080/3326 to review the following change. Change subject: WIP: KUDU-1466: improve error message when writes fail at TS .. WIP: KU

[kudu-CR] WIP: KUDU-1466: improve error message when writes fail at TS

2016-06-07 Thread Kudu Jenkins (Code Review)
Kudu Jenkins has posted comments on this change. Change subject: WIP: KUDU-1466: improve error message when writes fail at TS .. Patch Set 1: Build Started http://104.196.14.100/job/kudu-gerrit/1758/ -- To view, visit http://

[kudu-CR] KUDU-1353: remove per-tablet replica locations cache

2016-06-07 Thread Adar Dembo (Code Review)
Adar Dembo has posted comments on this change. Change subject: KUDU-1353: remove per-tablet replica locations cache .. Patch Set 8: (2 comments) http://gerrit.cloudera.org:8080/#/c/2887/8/src/kudu/master/catalog_manager.cc Fil

[kudu-CR] KUDU-1353: remove per-tablet replica locations cache

2016-06-07 Thread Adar Dembo (Code Review)
Hello Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/2887 to look at the new patch set (#9). Change subject: KUDU-1353: remove per-tablet replica locations cache .. KUDU-135

[kudu-CR] [java client] fix reruns of TestKuduTable.testGetLocations

2016-06-07 Thread Adar Dembo (Code Review)
Hello Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/3319 to look at the new patch set (#3). Change subject: [java client] fix reruns of TestKuduTable.testGetLocations .. [j

[kudu-CR] KUDU-1353: remove per-tablet replica locations cache

2016-06-07 Thread Kudu Jenkins (Code Review)
Kudu Jenkins has posted comments on this change. Change subject: KUDU-1353: remove per-tablet replica locations cache .. Patch Set 9: Build Started http://104.196.14.100/job/kudu-gerrit/1760/ -- To view, visit http://gerrit.c

[kudu-CR] KUDU-1473: fix some tablet lock usage in CatalogManager

2016-06-07 Thread Adar Dembo (Code Review)
Adar Dembo has posted comments on this change. Change subject: KUDU-1473: fix some tablet lock usage in CatalogManager .. Patch Set 2: (1 comment) http://gerrit.cloudera.org:8080/#/c/3309/2//COMMIT_MSG Commit Message: Line 15

[kudu-CR] KUDU-1473: fix some tablet lock usage in CatalogManager

2016-06-07 Thread Kudu Jenkins (Code Review)
Kudu Jenkins has posted comments on this change. Change subject: KUDU-1473: fix some tablet lock usage in CatalogManager .. Patch Set 3: Build Started http://104.196.14.100/job/kudu-gerrit/1761/ -- To view, visit http://gerri

[kudu-CR] [java client] fix reruns of TestKuduTable.testGetLocations

2016-06-07 Thread Kudu Jenkins (Code Review)
Kudu Jenkins has posted comments on this change. Change subject: [java client] fix reruns of TestKuduTable.testGetLocations .. Patch Set 3: Build Started http://104.196.14.100/job/kudu-gerrit/1759/ -- To view, visit http://ge

[kudu-CR] [java-client] disconnect all tablets in TestAsyncKuduSession.disconnectAndWait

2016-06-07 Thread Kudu Jenkins (Code Review)
Kudu Jenkins has posted comments on this change. Change subject: [java-client] disconnect all tablets in TestAsyncKuduSession.disconnectAndWait .. Patch Set 1: Build Started http://104.196.14.100/job/kudu-gerrit/1762/ -- To

[kudu-CR] [java-client] disconnect all tablets in TestAsyncKuduSession.disconnectAndWait

2016-06-07 Thread Dan Burkert (Code Review)
Hello Jean-Daniel Cryans, Adar Dembo, I'd like you to do a code review. Please visit http://gerrit.cloudera.org:8080/3329 to review the following change. Change subject: [java-client] disconnect all tablets in TestAsyncKuduSession.disconnectAndWait

[kudu-CR] [java-client] RPC timeout may sometimes be reported as max attemps violation

2016-06-07 Thread Kudu Jenkins (Code Review)
Kudu Jenkins has posted comments on this change. Change subject: [java-client] RPC timeout may sometimes be reported as max attemps violation .. Patch Set 1: Build Started http://104.196.14.100/job/kudu-gerrit/1763/ -- To vi

[kudu-CR] [java-client] RPC timeout may sometimes be reported as max attemps violation

2016-06-07 Thread Dan Burkert (Code Review)
Hello Jean-Daniel Cryans, Adar Dembo, I'd like you to do a code review. Please visit http://gerrit.cloudera.org:8080/3330 to review the following change. Change subject: [java-client] RPC timeout may sometimes be reported as max attemps violation ..

[kudu-CR] [java client] fix reruns of TestKuduTable.testGetLocations

2016-06-07 Thread Jean-Daniel Cryans (Code Review)
Jean-Daniel Cryans has submitted this change and it was merged. Change subject: [java client] fix reruns of TestKuduTable.testGetLocations .. [java client] fix reruns of TestKuduTable.testGetLocations The test expected a table

[kudu-CR] [java client] fix reruns of TestKuduTable.testGetLocations

2016-06-07 Thread Jean-Daniel Cryans (Code Review)
Jean-Daniel Cryans has posted comments on this change. Change subject: [java client] fix reruns of TestKuduTable.testGetLocations .. Patch Set 3: Code-Review+2 -- To view, visit http://gerrit.cloudera.org:8080/3319 To unsubscr

[kudu-CR] [java-client] RPC timeout may sometimes be reported as max attempts violation

2016-06-07 Thread Dan Burkert (Code Review)
Dan Burkert has uploaded a new patch set (#2). Change subject: [java-client] RPC timeout may sometimes be reported as max attempts violation .. [java-client] RPC timeout may sometimes be reported as max attempts violation Fixes

[kudu-CR] [java-client] RPC timeout may sometimes be reported as max attempts violation

2016-06-07 Thread Kudu Jenkins (Code Review)
Kudu Jenkins has posted comments on this change. Change subject: [java-client] RPC timeout may sometimes be reported as max attempts violation .. Patch Set 2: Build Started http://104.196.14.100/job/kudu-gerrit/1764/ -- To v

[kudu-CR] [java-client] RPC timeout may sometimes be reported as max attempts violation

2016-06-07 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: (1 comment) http://gerrit.cloudera.org:8080/#/c/3330/2/java/

[kudu-CR] [java-client] RPC timeout may sometimes be reported as max attempts violation

2016-06-07 Thread Dan Burkert (Code Review)
Dan Burkert has posted comments on this change. Change subject: [java-client] RPC timeout may sometimes be reported as max attempts violation .. Patch Set 2: (1 comment) http://gerrit.cloudera.org:8080/#/c/3330/2/java/kudu-cl

[kudu-CR] [java-client] RPC timeout may sometimes be reported as max attempts violation

2016-06-07 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: (1 comment) http://gerrit.cloudera.org:8080/#/c/3330/2/java/

[kudu-CR] [java-client] disconnect all tablets in TestAsyncKuduSession.disconnectAndWait

2016-06-07 Thread Kudu Jenkins (Code Review)
Kudu Jenkins has posted comments on this change. Change subject: [java-client] disconnect all tablets in TestAsyncKuduSession.disconnectAndWait .. Patch Set 1: -Verified Build Started http://104.196.14.100/job/kudu-gerrit/1765

[kudu-CR] [java-client] RPC timeout may sometimes be reported as max attempts violation

2016-06-07 Thread Dan Burkert (Code Review)
Dan Burkert has posted comments on this change. Change subject: [java-client] RPC timeout may sometimes be reported as max attempts violation .. Patch Set 2: (1 comment) http://gerrit.cloudera.org:8080/#/c/3330/2/java/kudu-cl

[kudu-CR] [java-client] RPC timeout may sometimes be reported as max attempts violation

2016-06-07 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: (1 comment) http://gerrit.cloudera.org:8080/#/c/3330/2/java/

[kudu-CR] [java-client] disconnect all tablets in TestAsyncKuduSession.disconnectAndWait

2016-06-07 Thread Jean-Daniel Cryans (Code Review)
Jean-Daniel Cryans has posted comments on this change. Change subject: [java-client] disconnect all tablets in TestAsyncKuduSession.disconnectAndWait .. Patch Set 1: (1 comment) Nice catch. http://gerrit.cloudera.org:8080/#/

[kudu-CR] [java-client] RPC timeout may sometimes be reported as max attempts violation

2016-06-07 Thread Kudu Jenkins (Code Review)
Kudu Jenkins has posted comments on this change. Change subject: [java-client] RPC timeout may sometimes be reported as max attempts violation .. Patch Set 3: Build Started http://104.196.14.100/job/kudu-gerrit/1766/ -- To v

[kudu-CR] [java-client] RPC timeout may sometimes be reported as max attempts violation

2016-06-07 Thread Dan Burkert (Code Review)
Hello Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/3330 to look at the new patch set (#3). Change subject: [java-client] RPC timeout may sometimes be reported as max attempts violation ..

[kudu-CR] [java-client] RPC timeout may sometimes be reported as max attempts violation

2016-06-07 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 3: Code-Review+2 -- To view, visit http://gerrit.cloudera.org:8

[kudu-CR] [java-client] RPC timeout may sometimes be reported as max attempts violation

2016-06-07 Thread Adar Dembo (Code Review)
Adar Dembo has posted comments on this change. Change subject: [java-client] RPC timeout may sometimes be reported as max attempts violation .. Patch Set 3: Why do we even bother limiting the number of attempts? The C++ client

[kudu-CR] [java-client] RPC timeout may sometimes be reported as max attempts violation

2016-06-07 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 3: > Why do we even bother limiting the number of attempts? The

[kudu-CR] [java-client] disconnect all tablets in TestAsyncKuduSession.disconnectAndWait

2016-06-07 Thread Kudu Jenkins (Code Review)
Kudu Jenkins has posted comments on this change. Change subject: [java-client] disconnect all tablets in TestAsyncKuduSession.disconnectAndWait .. Patch Set 2: Build Started http://104.196.14.100/job/kudu-gerrit/1767/ -- To

[kudu-CR] [java-client] disconnect all tablets in TestAsyncKuduSession.disconnectAndWait

2016-06-07 Thread Dan Burkert (Code Review)
Hello Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/3329 to look at the new patch set (#2). Change subject: [java-client] disconnect all tablets in TestAsyncKuduSession.disconnectAndWait .

[kudu-CR] [java-client] disconnect all tablets in TestAsyncKuduSession.disconnectAndWait

2016-06-07 Thread Dan Burkert (Code Review)
Dan Burkert has posted comments on this change. Change subject: [java-client] disconnect all tablets in TestAsyncKuduSession.disconnectAndWait .. Patch Set 1: (1 comment) http://gerrit.cloudera.org:8080/#/c/3329/1/java/kudu-c

[kudu-CR] [java-client] disconnect all tablets in TestAsyncKuduSession.disconnectAndWait

2016-06-07 Thread Jean-Daniel Cryans (Code Review)
Jean-Daniel Cryans has posted comments on this change. Change subject: [java-client] disconnect all tablets in TestAsyncKuduSession.disconnectAndWait .. Patch Set 2: (1 comment) http://gerrit.cloudera.org:8080/#/c/3329/2/java

[kudu-CR] [java-client] disconnect all tablets in TestAsyncKuduSession.disconnectAndWait

2016-06-07 Thread Dan Burkert (Code Review)
Dan Burkert has posted comments on this change. Change subject: [java-client] disconnect all tablets in TestAsyncKuduSession.disconnectAndWait .. Patch Set 2: (1 comment) http://gerrit.cloudera.org:8080/#/c/3329/2/java/kudu-c

[kudu-CR] [java-client] disconnect all tablets in TestAsyncKuduSession.disconnectAndWait

2016-06-07 Thread Jean-Daniel Cryans (Code Review)
Jean-Daniel Cryans has posted comments on this change. Change subject: [java-client] disconnect all tablets in TestAsyncKuduSession.disconnectAndWait .. Patch Set 2: (1 comment) http://gerrit.cloudera.org:8080/#/c/3329/2/java

[kudu-CR] [java-client] disconnect all tablets in TestAsyncKuduSession.disconnectAndWait

2016-06-07 Thread Dan Burkert (Code Review)
Dan Burkert has posted comments on this change. Change subject: [java-client] disconnect all tablets in TestAsyncKuduSession.disconnectAndWait .. Patch Set 2: (1 comment) http://gerrit.cloudera.org:8080/#/c/3329/2/java/kudu-c

[kudu-CR] [java-client] disconnect all tablets in TestAsyncKuduSession.disconnectAndWait

2016-06-07 Thread Jean-Daniel Cryans (Code Review)
Jean-Daniel Cryans has posted comments on this change. Change subject: [java-client] disconnect all tablets in TestAsyncKuduSession.disconnectAndWait .. Patch Set 2: Code-Review+2 (1 comment) http://gerrit.cloudera.org:8080/#

[kudu-CR] WIP: KUDU-1466: improve error message when writes fail at TS

2016-06-07 Thread Adar Dembo (Code Review)
Adar Dembo has posted comments on this change. Change subject: WIP: KUDU-1466: improve error message when writes fail at TS .. Patch Set 1: (1 comment) http://gerrit.cloudera.org:8080/#/c/3326/1//COMMIT_MSG Commit Message: PS

[kudu-CR] [java-client] disconnect all tablets in TestAsyncKuduSession.disconnectAndWait

2016-06-07 Thread Dan Burkert (Code Review)
Dan Burkert has posted comments on this change. Change subject: [java-client] disconnect all tablets in TestAsyncKuduSession.disconnectAndWait .. Patch Set 2: Code-Review-2 The test is now flaky. I think it's always been an i

[kudu-CR] update dist-test configuration for new client.py location

2016-06-07 Thread Dan Burkert (Code Review)
Hello Jean-Daniel Cryans, Adar Dembo, I'd like you to do a code review. Please visit http://gerrit.cloudera.org:8080/3332 to review the following change. Change subject: update dist-test configuration for new client.py location ..

[kudu-CR] update dist-test configuration for new client.py location

2016-06-07 Thread Kudu Jenkins (Code Review)
Kudu Jenkins has posted comments on this change. Change subject: update dist-test configuration for new client.py location .. Patch Set 1: Build Started http://104.196.14.100/job/kudu-gerrit/1770/ -- To view, visit http://ger

[kudu-CR] [java-client] disconnect all tablets in TestAsyncKuduSession.disconnectAndWait

2016-06-07 Thread Dan Burkert (Code Review)
Hello Jean-Daniel Cryans, Adar Dembo, Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/3329 to look at the new patch set (#3). Change subject: [java-client] disconnect all tablets in TestAsyncKuduSession.disconnectAndWait .

[kudu-CR] [java-client] disconnect all tablets in TestAsyncKuduSession.disconnectAndWait

2016-06-07 Thread Kudu Jenkins (Code Review)
Kudu Jenkins has posted comments on this change. Change subject: [java-client] disconnect all tablets in TestAsyncKuduSession.disconnectAndWait .. Patch Set 3: Build Started http://104.196.14.100/job/kudu-gerrit/1769/ -- To

[kudu-CR] [java-client] RPC timeout may sometimes be reported as max attempts violation

2016-06-07 Thread Kudu Jenkins (Code Review)
Kudu Jenkins has posted comments on this change. Change subject: [java-client] RPC timeout may sometimes be reported as max attempts violation .. Patch Set 5: Build Started http://104.196.14.100/job/kudu-gerrit/1771/ -- To v

[kudu-CR] [java-client] disconnect all tablets in TestAsyncKuduSession.disconnectAndWait

2016-06-07 Thread Kudu Jenkins (Code Review)
Kudu Jenkins has posted comments on this change. Change subject: [java-client] disconnect all tablets in TestAsyncKuduSession.disconnectAndWait .. Patch Set 4: Build Started http://104.196.14.100/job/kudu-gerrit/1772/ -- To

[kudu-CR] [java-client] RPC timeout may sometimes be reported as max attempts violation

2016-06-07 Thread Kudu Jenkins (Code Review)
Kudu Jenkins has posted comments on this change. Change subject: [java-client] RPC timeout may sometimes be reported as max attempts violation .. Patch Set 4: Build Started http://104.196.14.100/job/kudu-gerrit/1768/ -- To v

[kudu-CR] update dist-test configuration for new client.py location

2016-06-07 Thread Kudu Jenkins (Code Review)
Kudu Jenkins has posted comments on this change. Change subject: update dist-test configuration for new client.py location .. Patch Set 2: Build Started http://104.196.14.100/job/kudu-gerrit/1773/ -- To view, visit http://ger

[kudu-CR] update dist-test configuration for new client.py location

2016-06-07 Thread Kudu Jenkins (Code Review)
Kudu Jenkins has posted comments on this change. Change subject: update dist-test configuration for new client.py location .. Patch Set 3: Build Started http://104.196.14.100/job/kudu-gerrit/1774/ -- To view, visit http://ger

[kudu-CR] update dist-test configuration for new client.py location

2016-06-07 Thread Dan Burkert (Code Review)
Hello Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/3332 to look at the new patch set (#3). Change subject: update dist-test configuration for new client.py location .. upd

[kudu-CR] KUDU-1363: Add in-list predicates for extracting a set of equalities.

2016-06-07 Thread Adar Dembo (Code Review)
Adar Dembo has posted comments on this change. Change subject: KUDU-1363: Add in-list predicates for extracting a set of equalities. .. Patch Set 5: (20 comments) OK, finished reviewing everything except some of the hairier p

[kudu-CR] update dist-test configuration for new client.py location

2016-06-07 Thread Adar Dembo (Code Review)
Adar Dembo has posted comments on this change. Change subject: update dist-test configuration for new client.py location .. Patch Set 3: Code-Review+2 -- To view, visit http://gerrit.cloudera.org:8080/3332 To unsubscribe, visi

[kudu-CR] [java-client] RPC timeout may sometimes be reported as max attempts violation

2016-06-07 Thread Kudu Jenkins (Code Review)
Kudu Jenkins has posted comments on this change. Change subject: [java-client] RPC timeout may sometimes be reported as max attempts violation .. Patch Set 6: Build Started http://104.196.14.100/job/kudu-gerrit/1775/ -- To v

[kudu-CR] [java-client] disconnect all tablets in TestAsyncKuduSession.disconnectAndWait

2016-06-07 Thread Kudu Jenkins (Code Review)
Kudu Jenkins has posted comments on this change. Change subject: [java-client] disconnect all tablets in TestAsyncKuduSession.disconnectAndWait .. Patch Set 5: Build Started http://104.196.14.100/job/kudu-gerrit/1776/ -- To

[kudu-CR] update dist-test configuration for new client.py location

2016-06-07 Thread Dan Burkert (Code Review)
Dan Burkert has submitted this change and it was merged. Change subject: update dist-test configuration for new client.py location .. update dist-test configuration for new client.py location client.py has been moved to bin/cli

[kudu-CR] KUDU-1473: fix some tablet lock usage in CatalogManager

2016-06-07 Thread Adar Dembo (Code Review)
Hello Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/3309 to look at the new patch set (#4). Change subject: KUDU-1473: fix some tablet lock usage in CatalogManager .. KUDU-

[kudu-CR] KUDU-1473: fix some tablet lock usage in CatalogManager

2016-06-07 Thread Kudu Jenkins (Code Review)
Kudu Jenkins has posted comments on this change. Change subject: KUDU-1473: fix some tablet lock usage in CatalogManager .. Patch Set 4: Build Started http://104.196.14.100/job/kudu-gerrit/1777/ -- To view, visit http://gerri

[kudu-CR] [java-client] disconnect all tablets in TestAsyncKuduSession.disconnectAndWait

2016-06-07 Thread Dan Burkert (Code Review)
Dan Burkert has posted comments on this change. Change subject: [java-client] disconnect all tablets in TestAsyncKuduSession.disconnectAndWait .. Patch Set 5: -Code-Review -- To view, visit http://gerrit.cloudera.org:8080/332

[kudu-CR] KUDU-1353: remove per-tablet replica locations cache

2016-06-07 Thread Dan Burkert (Code Review)
Dan Burkert has posted comments on this change. Change subject: KUDU-1353: remove per-tablet replica locations cache .. Patch Set 9: (1 comment) http://gerrit.cloudera.org:8080/#/c/2887/9/src/kudu/master/master-path-handlers.c

[kudu-CR] [java-client] RPC timeout may sometimes be reported as max attempts violation

2016-06-07 Thread Dan Burkert (Code Review)
Dan Burkert has submitted this change and it was merged. Change subject: [java-client] RPC timeout may sometimes be reported as max attempts violation .. [java-client] RPC timeout may sometimes be reported as max attempts viola

[kudu-CR] [java-client] RPC timeout may sometimes be reported as max attempts violation

2016-06-07 Thread Dan Burkert (Code Review)
Dan Burkert has posted comments on this change. Change subject: [java-client] RPC timeout may sometimes be reported as max attempts violation .. Patch Set 6: Code-Review+2 -- To view, visit http://gerrit.cloudera.org:8080/333

[kudu-CR] [java-client] disconnect all tablets in TestAsyncKuduSession.disconnectAndWait

2016-06-07 Thread Jean-Daniel Cryans (Code Review)
Jean-Daniel Cryans has posted comments on this change. Change subject: [java-client] disconnect all tablets in TestAsyncKuduSession.disconnectAndWait .. Patch Set 5: Code-Review+2 -- To view, visit http://gerrit.cloudera.org:

[kudu-CR] [java-client] disconnect all tablets in TestAsyncKuduSession.disconnectAndWait

2016-06-07 Thread Jean-Daniel Cryans (Code Review)
Jean-Daniel Cryans has submitted this change and it was merged. Change subject: [java-client] disconnect all tablets in TestAsyncKuduSession.disconnectAndWait .. [java-client] disconnect all tablets in TestAsyncKuduSession.disc

[kudu-CR] KUDU-1353: remove per-tablet replica locations cache

2016-06-07 Thread Adar Dembo (Code Review)
Adar Dembo has posted comments on this change. Change subject: KUDU-1353: remove per-tablet replica locations cache .. Patch Set 9: (1 comment) http://gerrit.cloudera.org:8080/#/c/2887/9/src/kudu/master/master-path-handlers.cc

[kudu-CR] KUDU-1473: fix some tablet lock usage in CatalogManager

2016-06-07 Thread Adar Dembo (Code Review)
Hello Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/3309 to look at the new patch set (#5). Change subject: KUDU-1473: fix some tablet lock usage in CatalogManager .. KUDU-

[kudu-CR] ts itest-base.h: wait for bootstrapping to finish when waiting for replicas

2016-06-07 Thread Kudu Jenkins (Code Review)
Kudu Jenkins has posted comments on this change. Change subject: ts_itest-base.h: wait for bootstrapping to finish when waiting for replicas .. Patch Set 3: Build Started http://104.196.14.100/job/kudu-gerrit/1780/ -- To vie

[kudu-CR] KUDU-1353: remove per-tablet replica locations cache

2016-06-07 Thread Kudu Jenkins (Code Review)
Kudu Jenkins has posted comments on this change. Change subject: KUDU-1353: remove per-tablet replica locations cache .. Patch Set 10: Build Started http://104.196.14.100/job/kudu-gerrit/1778/ -- To view, visit http://gerrit.

[kudu-CR] KUDU-1473: fix some tablet lock usage in CatalogManager

2016-06-07 Thread Kudu Jenkins (Code Review)
Kudu Jenkins has posted comments on this change. Change subject: KUDU-1473: fix some tablet lock usage in CatalogManager .. Patch Set 5: Build Started http://104.196.14.100/job/kudu-gerrit/1779/ -- To view, visit http://gerri

[kudu-CR] KUDU-1353: remove per-tablet replica locations cache

2016-06-07 Thread Adar Dembo (Code Review)
Hello Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/2887 to look at the new patch set (#10). Change subject: KUDU-1353: remove per-tablet replica locations cache .. KUDU-13

[kudu-CR] ts itest-base.h: wait for bootstrapping to finish when waiting for replicas

2016-06-07 Thread Adar Dembo (Code Review)
Adar Dembo has posted comments on this change. Change subject: ts_itest-base.h: wait for bootstrapping to finish when waiting for replicas .. Patch Set 3: Verified+1 Overriding Jenkins, the one failure is a flaky test fixed in

[kudu-CR] KUDU-1353: remove per-tablet replica locations cache

2016-06-07 Thread Adar Dembo (Code Review)
Adar Dembo has posted comments on this change. Change subject: KUDU-1353: remove per-tablet replica locations cache .. Patch Set 10: Verified+1 Overriding Jenkins, the one failure was a Java test (testDisconnect) in TSAN mode.

[kudu-CR] KUDU-1473: fix some tablet lock usage in CatalogManager

2016-06-07 Thread Adar Dembo (Code Review)
Adar Dembo has posted comments on this change. Change subject: KUDU-1473: fix some tablet lock usage in CatalogManager .. Patch Set 5: Verified+1 Overriding Jenkins, the one failure was a Java test (testDisconnect) in TSAN mod