Mike Percy has posted comments on this change. ( http://gerrit.cloudera.org:8080/9246 )
Change subject: KUDU-2274. consensus: Remove ConsensusMetadata thread safety ...................................................................... Patch Set 1: (3 comments) http://gerrit.cloudera.org:8080/#/c/9246/1/src/kudu/consensus/consensus_meta.h File src/kudu/consensus/consensus_meta.h: PS1: > Is this just a WIP patch? Well, semi WIP, I wanted your opinion but I tend to agree that we should restructure the class and use a recursive fake mutex (per the commit message). http://gerrit.cloudera.org:8080/#/c/9246/1/src/kudu/consensus/consensus_meta.h@236 PS1, Line 236: TODO(mpercy) Doc this > Indeed: it would be nice to explain the reasoning behind having this fake m oops, will do http://gerrit.cloudera.org:8080/#/c/9246/1/src/kudu/consensus/consensus_meta.h@237 PS1, Line 237: lock_ > nit: maybe, rename it to fake_lock_? good idea -- To view, visit http://gerrit.cloudera.org:8080/9246 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: kudu Gerrit-Branch: master Gerrit-MessageType: comment Gerrit-Change-Id: Ia8d086c3fba52826ebe0d3a44842d53ecb6a9265 Gerrit-Change-Number: 9246 Gerrit-PatchSet: 1 Gerrit-Owner: Mike Percy <[email protected]> Gerrit-Reviewer: Alexey Serbin <[email protected]> Gerrit-Reviewer: Kudu Jenkins Gerrit-Reviewer: Mike Percy <[email protected]> Gerrit-Comment-Date: Wed, 07 Feb 2018 23:04:00 +0000 Gerrit-HasComments: Yes
