[devel] [PATCH 0 of 1] Review Request for IMM: Use existing PBE mutation record to guard against concurrent class create/delete [#757]

2014-02-21 Thread Anders Bjornerstedt
Summary: IMM: Use existing PBE mutation record to guard against concurrent class create/delete [#757] Review request for Trac Ticket(s): 757 Peer Reviewer(s): Neel Pull request to: Affected branch(es): 4.2(?); 4.3; 4.4: default(4.5) Development branch: Impacted

Re: [devel] [PATCH 0 of 1] Review Request for IMM: Use existing PBE mutation record to guard against concurrent class create/delete [#757]

2014-03-03 Thread Neelakanta Reddy
Hi AndersBj, Reviewed and tested the patch. Ack with nimor comment. comment: The code for ImmModel::classDelete in the patch may not hit because, In classDelete the class name will be erased once it is marked for deletion. and ERR_NOT_EXIST will be returned(The following code will hit). if (

Re: [devel] [PATCH 0 of 1] Review Request for IMM: Use existing PBE mutation record to guard against concurrent class create/delete [#757]

2014-03-04 Thread Anders Bjornerstedt
Neelakanta Reddy wrote: > Hi AndersBj, > > > Reviewed and tested the patch. > Ack with nimor comment. > > comment: > > The code for ImmModel::classDelete in the patch may not hit because, > In classDelete the class name will be erased once it is marked for > deletion. and ERR_NOT_EXIST will be ret