song bruce zhang has posted comments on this change. Change subject: KUDU-1436. Concurrent remote bootstrap session starts can crash or corrupt tablets ......................................................................
Patch Set 1: (1 comment) http://gerrit.cloudera.org:8080/#/c/2913/1/src/kudu/tserver/remote_bootstrap_session.cc File src/kudu/tserver/remote_bootstrap_session.cc: Line 74: std As have already removed calling "session->init" in else of BeginRemoteBootstrapSession in remote_bootstrap_service.cc What the "if(!l.owns_lock())" and "if (initted_)" used for? Is it a double check or some other consideration? -- To view, visit http://gerrit.cloudera.org:8080/2913 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-MessageType: comment Gerrit-Change-Id: I8ec55e7b51da9e7c795c18146f04d539d1e44ccd Gerrit-PatchSet: 1 Gerrit-Project: kudu Gerrit-Branch: master Gerrit-Owner: Todd Lipcon <[email protected]> Gerrit-Reviewer: Kudu Jenkins Gerrit-Reviewer: Mike Percy <[email protected]> Gerrit-Reviewer: song bruce zhang <[email protected]> Gerrit-HasComments: Yes
