[kudu-CR] master: add assert checks for leader lock
Adar Dembo has submitted this change and it was merged. Change subject: master: add assert checks for leader_lock .. master: add assert checks for leader_lock A side effect of recursive checking in RWMutex is that we can now assert that a RWMutex is held for reading/writing. Let's add that to the various catalog manager entry points. Change-Id: Iefb5762c70192b27490cc71e20568815d18d6ad5 Reviewed-on: http://gerrit.cloudera.org:8080/3642 Tested-by: Kudu Jenkins Reviewed-by: Dan Burkert --- M src/kudu/client/client-test.cc M src/kudu/integration-tests/alter_table-test.cc M src/kudu/integration-tests/create-table-stress-test.cc M src/kudu/integration-tests/mini_cluster.cc M src/kudu/integration-tests/mini_cluster.h M src/kudu/master/catalog_manager.cc M src/kudu/master/catalog_manager.h M src/kudu/master/master-test-util.h M src/kudu/master/master.cc 9 files changed, 78 insertions(+), 41 deletions(-) Approvals: Dan Burkert: Looks good to me, approved Kudu Jenkins: Verified -- To view, visit http://gerrit.cloudera.org:8080/3642 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-MessageType: merged Gerrit-Change-Id: Iefb5762c70192b27490cc71e20568815d18d6ad5 Gerrit-PatchSet: 6 Gerrit-Project: kudu Gerrit-Branch: master Gerrit-Owner: Adar Dembo Gerrit-Reviewer: Adar Dembo Gerrit-Reviewer: Dan Burkert Gerrit-Reviewer: Kudu Jenkins Gerrit-Reviewer: Todd Lipcon
[kudu-CR] master: add assert checks for leader lock
Dan Burkert has posted comments on this change. Change subject: master: add assert checks for leader_lock .. Patch Set 5: Code-Review+2 -- To view, visit http://gerrit.cloudera.org:8080/3642 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-MessageType: comment Gerrit-Change-Id: Iefb5762c70192b27490cc71e20568815d18d6ad5 Gerrit-PatchSet: 5 Gerrit-Project: kudu Gerrit-Branch: master Gerrit-Owner: Adar Dembo Gerrit-Reviewer: Adar Dembo Gerrit-Reviewer: Dan Burkert Gerrit-Reviewer: Kudu Jenkins Gerrit-Reviewer: Todd Lipcon Gerrit-HasComments: No
[kudu-CR] master: add assert checks for leader lock
Kudu Jenkins has posted comments on this change. Change subject: master: add assert checks for leader_lock .. Patch Set 5: Build Started http://104.196.14.100/job/kudu-gerrit/2474/ -- To view, visit http://gerrit.cloudera.org:8080/3642 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-MessageType: comment Gerrit-Change-Id: Iefb5762c70192b27490cc71e20568815d18d6ad5 Gerrit-PatchSet: 5 Gerrit-Project: kudu Gerrit-Branch: master Gerrit-Owner: Adar Dembo Gerrit-Reviewer: Adar Dembo Gerrit-Reviewer: Dan Burkert Gerrit-Reviewer: Kudu Jenkins Gerrit-Reviewer: Todd Lipcon Gerrit-HasComments: No
[kudu-CR] master: add assert checks for leader lock
Adar Dembo has posted comments on this change. Change subject: master: add assert checks for leader_lock .. Patch Set 4: Verified+1 Another chrpath() failure in dist-test. -- To view, visit http://gerrit.cloudera.org:8080/3642 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-MessageType: comment Gerrit-Change-Id: Iefb5762c70192b27490cc71e20568815d18d6ad5 Gerrit-PatchSet: 4 Gerrit-Project: kudu Gerrit-Branch: master Gerrit-Owner: Adar Dembo Gerrit-Reviewer: Adar Dembo Gerrit-Reviewer: Dan Burkert Gerrit-Reviewer: Kudu Jenkins Gerrit-Reviewer: Todd Lipcon Gerrit-HasComments: No
[kudu-CR] master: add assert checks for leader lock
Dan Burkert has posted comments on this change. Change subject: master: add assert checks for leader_lock .. Patch Set 4: Code-Review+2 -- To view, visit http://gerrit.cloudera.org:8080/3642 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-MessageType: comment Gerrit-Change-Id: Iefb5762c70192b27490cc71e20568815d18d6ad5 Gerrit-PatchSet: 4 Gerrit-Project: kudu Gerrit-Branch: master Gerrit-Owner: Adar Dembo Gerrit-Reviewer: Adar Dembo Gerrit-Reviewer: Dan Burkert Gerrit-Reviewer: Kudu Jenkins Gerrit-Reviewer: Todd Lipcon Gerrit-HasComments: No
[kudu-CR] master: add assert checks for leader lock
Kudu Jenkins has posted comments on this change. Change subject: master: add assert checks for leader_lock .. Patch Set 4: Build Started http://104.196.14.100/job/kudu-gerrit/2459/ -- To view, visit http://gerrit.cloudera.org:8080/3642 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-MessageType: comment Gerrit-Change-Id: Iefb5762c70192b27490cc71e20568815d18d6ad5 Gerrit-PatchSet: 4 Gerrit-Project: kudu Gerrit-Branch: master Gerrit-Owner: Adar Dembo Gerrit-Reviewer: Adar Dembo Gerrit-Reviewer: Dan Burkert Gerrit-Reviewer: Kudu Jenkins Gerrit-Reviewer: Todd Lipcon Gerrit-HasComments: No
[kudu-CR] master: add assert checks for leader lock
Kudu Jenkins has posted comments on this change. Change subject: master: add assert checks for leader_lock .. Patch Set 3: Build Started http://104.196.14.100/job/kudu-gerrit/2447/ -- To view, visit http://gerrit.cloudera.org:8080/3642 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-MessageType: comment Gerrit-Change-Id: Iefb5762c70192b27490cc71e20568815d18d6ad5 Gerrit-PatchSet: 3 Gerrit-Project: kudu Gerrit-Branch: master Gerrit-Owner: Adar Dembo Gerrit-Reviewer: Adar Dembo Gerrit-Reviewer: Dan Burkert Gerrit-Reviewer: Kudu Jenkins Gerrit-Reviewer: Todd Lipcon Gerrit-HasComments: No
[kudu-CR] master: add assert checks for leader lock
Dan Burkert has posted comments on this change. Change subject: master: add assert checks for leader_lock .. Patch Set 2: Code-Review+2 -- To view, visit http://gerrit.cloudera.org:8080/3642 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-MessageType: comment Gerrit-Change-Id: Iefb5762c70192b27490cc71e20568815d18d6ad5 Gerrit-PatchSet: 2 Gerrit-Project: kudu Gerrit-Branch: master Gerrit-Owner: Adar Dembo Gerrit-Reviewer: Adar Dembo Gerrit-Reviewer: Dan Burkert Gerrit-Reviewer: Todd Lipcon Gerrit-HasComments: No
[kudu-CR] master: add assert checks for leader lock
Adar Dembo has posted comments on this change. Change subject: master: add assert checks for leader_lock .. Patch Set 2: Verified+1 Isolate crashed. -- To view, visit http://gerrit.cloudera.org:8080/3642 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-MessageType: comment Gerrit-Change-Id: Iefb5762c70192b27490cc71e20568815d18d6ad5 Gerrit-PatchSet: 2 Gerrit-Project: kudu Gerrit-Branch: master Gerrit-Owner: Adar Dembo Gerrit-Reviewer: Adar Dembo Gerrit-Reviewer: Dan Burkert Gerrit-Reviewer: Kudu Jenkins Gerrit-Reviewer: Todd Lipcon Gerrit-HasComments: No
[kudu-CR] master: add assert checks for leader lock
Kudu Jenkins has posted comments on this change. Change subject: master: add assert checks for leader_lock .. Patch Set 2: Build Started http://104.196.14.100/job/kudu-gerrit/2420/ -- To view, visit http://gerrit.cloudera.org:8080/3642 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-MessageType: comment Gerrit-Change-Id: Iefb5762c70192b27490cc71e20568815d18d6ad5 Gerrit-PatchSet: 2 Gerrit-Project: kudu Gerrit-Branch: master Gerrit-Owner: Adar Dembo Gerrit-Reviewer: Dan Burkert Gerrit-Reviewer: Kudu Jenkins Gerrit-Reviewer: Todd Lipcon Gerrit-HasComments: No
[kudu-CR] master: add assert checks for leader lock
Adar Dembo has uploaded a new change for review. http://gerrit.cloudera.org:8080/3642 Change subject: master: add assert checks for leader_lock .. master: add assert checks for leader_lock A side effect of recursive checking in RWMutex is that we can now assert that a RWMutex is held for reading/writing. Let's add that to the various catalog manager entry points. Change-Id: Iefb5762c70192b27490cc71e20568815d18d6ad5 --- M src/kudu/client/client-test.cc M src/kudu/integration-tests/alter_table-test.cc M src/kudu/integration-tests/create-table-stress-test.cc M src/kudu/integration-tests/mini_cluster.cc M src/kudu/integration-tests/mini_cluster.h M src/kudu/master/catalog_manager.cc M src/kudu/master/catalog_manager.h M src/kudu/master/master-test-util.h M src/kudu/master/master.cc 9 files changed, 78 insertions(+), 41 deletions(-) git pull ssh://gerrit.cloudera.org:29418/kudu refs/changes/42/3642/1 -- To view, visit http://gerrit.cloudera.org:8080/3642 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-MessageType: newchange Gerrit-Change-Id: Iefb5762c70192b27490cc71e20568815d18d6ad5 Gerrit-PatchSet: 1 Gerrit-Project: kudu Gerrit-Branch: master Gerrit-Owner: Adar Dembo
[kudu-CR] master: add assert checks for leader lock
Kudu Jenkins has posted comments on this change. Change subject: master: add assert checks for leader_lock .. Patch Set 1: Build Started http://104.196.14.100/job/kudu-gerrit/2408/ -- To view, visit http://gerrit.cloudera.org:8080/3642 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-MessageType: comment Gerrit-Change-Id: Iefb5762c70192b27490cc71e20568815d18d6ad5 Gerrit-PatchSet: 1 Gerrit-Project: kudu Gerrit-Branch: master Gerrit-Owner: Adar Dembo Gerrit-Reviewer: Kudu Jenkins Gerrit-HasComments: No