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

2016-06-06 Thread David Ribeiro Alves (Code Review)
David Ribeiro Alves 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/

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

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

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

2016-06-06 Thread David Ribeiro Alves (Code Review)
David Ribeiro Alves has posted comments on this change. Change subject: Add weekly update for 06/06 .. Patch Set 1: (4 comments) http://gerrit.cloudera.org:8080/#/c/3316/1/_posts/2016-06-06-weekly-update.md File _posts/2016-06

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

2016-06-06 Thread Dan Burkert (Code Review)
Dan Burkert has posted comments on this change. Change subject: Add weekly update for 06/06 .. Patch Set 1: Code-Review+1 LGTM modulo David's comments. -- To view, visit http://gerrit.cloudera.org:8080/3316 To unsubscribe, vi

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

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

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

2016-06-06 Thread Jean-Daniel Cryans (Code Review)
Hello Dan Burkert, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/3316 to look at the new patch set (#2). Change subject: Add weekly update for 06/06 .. Add weekly update for 06/06 Chang

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

2016-06-06 Thread David Ribeiro Alves (Code Review)
David Ribeiro Alves has posted comments on this change. Change subject: Add weekly update for 06/06 .. Patch Set 2: Code-Review+2 (1 comment) lgtm, fix the nit if you think it's worth it. http://gerrit.cloudera.org:8080/#/c/3

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

2016-06-06 Thread Dan Burkert (Code Review)
Dan Burkert has posted comments on this change. Change subject: Add weekly update for 06/06 .. Patch Set 2: (1 comment) http://gerrit.cloudera.org:8080/#/c/3316/2/_posts/2016-06-06-weekly-update.md File _posts/2016-06-06-weekl

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

2016-06-06 Thread Jean-Daniel Cryans (Code Review)
Hello Dan Burkert, David Ribeiro Alves, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/3316 to look at the new patch set (#3). Change subject: Add weekly update for 06/06 .. Add weekly up

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

2016-06-06 Thread David Ribeiro Alves (Code Review)
David Ribeiro Alves has posted comments on this change. Change subject: Add weekly update for 06/06 .. Patch Set 4: Code-Review+2 -- To view, visit http://gerrit.cloudera.org:8080/3316 To unsubscribe, visit http://gerrit.cloud

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

2016-06-06 Thread Dan Burkert (Code Review)
Dan Burkert has posted comments on this change. Change subject: Add weekly update for 06/06 .. Patch Set 4: Code-Review+2 -- To view, visit http://gerrit.cloudera.org:8080/3316 To unsubscribe, visit http://gerrit.cloudera.org:

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

2016-06-06 Thread Jean-Daniel Cryans (Code Review)
Hello Dan Burkert, David Ribeiro Alves, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/3316 to look at the new patch set (#4). Change subject: Add weekly update for 06/06 .. Add weekly up

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

2016-06-06 Thread Dan Burkert (Code Review)
Dan Burkert has posted comments on this change. Change subject: Add weekly update for 06/06 .. Patch Set 4: Verified+1 -- To view, visit http://gerrit.cloudera.org:8080/3316 To unsubscribe, visit http://gerrit.cloudera.org:808

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

2016-06-06 Thread Dan Burkert (Code Review)
Dan Burkert has submitted this change and it was merged. Change subject: Add weekly update for 06/06 .. Add weekly update for 06/06 Change-Id: I6814ef94ab522da5d1224fc5d86f975c3978c062 Reviewed-on: http://gerrit.cloudera.org:80

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

2016-06-06 Thread Mike Percy (Code Review)
Mike Percy has posted comments on this change. Change subject: Add weekly update for 06/06 .. Patch Set 5: (1 comment) http://gerrit.cloudera.org:8080/#/c/3316/5/_posts/2016-06-06-weekly-update.md File _posts/2016-06-06-weekly

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

2016-06-06 Thread Will Berkeley (Code Review)
Hello Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/3304 to look at the new patch set (#4). Change subject: KUDU-1398 CFile index blocks can store shortest separating prefix ..

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

2016-06-06 Thread Kudu Jenkins (Code Review)
Kudu Jenkins has posted comments on this change. Change subject: KUDU-1398 CFile index blocks can store shortest separating prefix .. Patch Set 4: Build Started http://104.196.14.100/job/kudu-gerrit/1749/ -- To view, visit h

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

2016-06-06 Thread Adar Dembo (Code Review)
Hello Dan Burkert, Jean-Daniel Cryans, I'd like you to do a code review. Please visit http://gerrit.cloudera.org:8080/3319 to review the following change. Change subject: [java client] fix reruns of TestKuduTable.testGetLocations

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

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

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

2016-06-06 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 1: (3 comments) http://gerrit.cloudera.org:8080/#/c/3308/1/src/kudu/inte

[kudu-CR] Improve tpch usability

2016-06-06 Thread Mike Percy (Code Review)
Mike Percy has posted comments on this change. Change subject: Improve tpch usability .. Patch Set 3: Verified+1 -- To view, visit http://gerrit.cloudera.org:8080/3279 To unsubscribe, visit http://gerrit.cloudera.org:8080/sett

[kudu-CR] Improve tpch usability

2016-06-06 Thread Mike Percy (Code Review)
Mike Percy has posted comments on this change. Change subject: Improve tpch usability .. Patch Set 3: Overriding Jenkins due to flaky test -- To view, visit http://gerrit.cloudera.org:8080/3279 To unsubscribe, visit http://ge

[kudu-CR] Improve tpch usability

2016-06-06 Thread Mike Percy (Code Review)
Mike Percy has submitted this change and it was merged. Change subject: Improve tpch usability .. Improve tpch usability * tpch.sh should use $ROOT to find the CPU scaling script (I had a problem with it otherwise) * tpch1.cc

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

2016-06-06 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 1: (3 comments) http://gerrit.cloudera.org:8080/#/c/3308/1/src/kudu/integ

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

2016-06-06 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 2: Build Started http://104.196.14.100/job/kudu-gerrit/1754/ -- To vie

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

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

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

2016-06-06 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 8: Build Started http://104.196.14.100/job/kudu-gerrit/1752/ -- To view, visit http://gerrit.c

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

2016-06-06 Thread Adar Dembo (Code Review)
Adar Dembo has uploaded a new patch set (#2). Change subject: ts_itest-base.h: wait for bootstrapping to finish when waiting for replicas .. ts_itest-base.h: wait for bootstrapping to finish when waiting for replicas WaitForRep

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

2016-06-06 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 2: Build Started http://104.196.14.100/job/kudu-gerrit/1753/ -- To view, visit http://gerri

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

2016-06-06 Thread Dan Burkert (Code Review)
Dan Burkert has posted comments on this change. Change subject: [java client] fix reruns of TestKuduTable.testGetLocations .. Patch Set 2: (1 comment) LGTM, just the small nit. http://gerrit.cloudera.org:8080/#/c/3319/2//COMM

[kudu-CR] KUDU-1477. Pending COMMIT message for failed write operation can prevent tablet startup

2016-06-06 Thread Todd Lipcon (Code Review)
Hello David Ribeiro Alves, I'd like you to do a code review. Please visit http://gerrit.cloudera.org:8080/3321 to review the following change. Change subject: KUDU-1477. Pending COMMIT message for failed write operation can prevent tablet startup ..

[kudu-CR] KUDU-1477. Pending COMMIT message for failed write operation can prevent tablet startup

2016-06-06 Thread Kudu Jenkins (Code Review)
Kudu Jenkins has posted comments on this change. Change subject: KUDU-1477. Pending COMMIT message for failed write operation can prevent tablet startup .. Patch Set 1: Build Started http://104.196.14.100/job/kudu-gerrit/1755/