On 07/18/2012 03:27 PM, Liu Yuan wrote: > On 07/18/2012 03:25 PM, levin li wrote: >>>>>>>> Should this be explained in the source code where vdi_exist() is >>>>>>>> called? >>>> >>>> How do you think about this? >>>> >> As I said in the commit log, before the notify message arrives at every node, >> a new cluster request adding the VDI just deleted may be issued, >> when the notify message arrives, if we try to delete the object list entries >> belong to this VDI regardless of whether the VDI exists, it may causes fatal >> error when next recovery happens. >> > > I guess Kazutaka means documenting this rationale in the source. > > Thanks, > Yuan > Oops, I misunderstood Kazutaka, I'll commit this in source code. thanks, levin |