Hey - So, I'm not getting exactly what you're saying here. I think you're saying that it's okay as is, but not sure. The implementation and the TD_Datastore method that I use don't have any need for rollback, although I could add some rollback=YES should the alloc/init of the CDTDocumentRevisions fail.