[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

[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

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

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

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

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

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

[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

[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

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

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

[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

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

[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

[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

[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

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

[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

[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

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

[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

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

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

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

[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

[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

[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

[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