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

2016-06-09 Thread Adar Dembo (Code Review)
Adar Dembo has submitted this change and it was merged. Change subject: KUDU-1473: fix some tablet lock usage in CatalogManager .. KUDU-1473: fix some tablet lock usage in CatalogManager This was probably due to the refactoring

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

2016-06-09 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 7: Verified+1 Overriding Jenkins, another Java testDisconnect failure. -- To view, visit http

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

2016-06-09 Thread Mike Percy (Code Review)
Mike Percy has posted comments on this change. Change subject: KUDU-1473: fix some tablet lock usage in CatalogManager .. Patch Set 7: Code-Review+2 -- To view, visit http://gerrit.cloudera.org:8080/3309 To unsubscribe, visit

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

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

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

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

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

2016-06-09 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 6: (1 comment) > (1 comment) > > I wish there was an easier way to hide the data object so

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

2016-06-09 Thread Mike Percy (Code Review)
Mike Percy has posted comments on this change. Change subject: KUDU-1473: fix some tablet lock usage in CatalogManager .. Patch Set 6: (1 comment) I wish there was an easier way to hide the data object so that we can only acc

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

2016-06-09 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 6: Verified+1 Another Java testDisconnect failure. -- To view, visit http://gerrit.cloudera.o

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

2016-06-08 Thread Jean-Daniel Cryans (Code Review)
Jean-Daniel Cryans has posted comments on this change. Change subject: KUDU-1473: fix some tablet lock usage in CatalogManager .. Patch Set 6: Code-Review+2 -- To view, visit http://gerrit.cloudera.org:8080/3309 To unsubscribe

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

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

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

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

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

2016-06-08 Thread Adar Dembo (Code Review)
Adar Dembo has uploaded a new patch set (#6). Change subject: KUDU-1473: fix some tablet lock usage in CatalogManager .. KUDU-1473: fix some tablet lock usage in CatalogManager This was probably due to the refactoring done in co

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

2016-06-08 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: (1 comment) http://gerrit.cloudera.org:8080/#/c/3309/5//COMMIT_MSG Commit Message: PS5, Li

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

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

[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

[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-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] 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] 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 3: Build Started http://104.196.14.100/job/kudu-gerrit/1761/ -- To view, visit http://gerri

[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 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-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] KUDU-1473: fix some tablet lock usage in CatalogManager

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

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

2016-06-03 Thread Adar Dembo (Code Review)
Hello Todd Lipcon, I'd like you to do a code review. Please visit http://gerrit.cloudera.org:8080/3309 to review the following change. Change subject: KUDU-1473: fix some tablet lock usage in CatalogManager .. KUDU-1473: