Interesting. If you know what you are doing you don't need to subsequently use CkpXeq. The idea here is to protect the index of the zipfile. If we overwrite that then it gets checkpointed. After that it's all appends and checkpointing isn't explcitly necessary (assuming the whole index was overwritten). Upon failure all changes are discarded and the saved data is restored. I suppose we should independently verify that this actually is what is happening. On Fri, 5 May 2023, David Smith wrote: > (I just made a force-push, only changed a typo in code comment). > > Thank you. I just checked that it does appear to work on a server with checkpointing enabled (in the no error case); infact appending against an existing archive fails against a server without checkpointing enabled. But, following on from your comment, we could verify more. e.g. that the checkpointing is giving the protection we want: keeping the archive valid if there's a problem during addition of files (I saw we don't use the kXR_ckpXeq->kXR_write sequenced version of write during CloseArchive and I'm not sure if that will properly preserve integrity if the checkpoint changes are discarded; and I didn't try to change that in this PR). > > I'll suggest to Guilherme to merge, so we can address #2004 now. > > -- > Reply to this email directly or view it on GitHub: > https://github.com/xrootd/xrootd/pull/2006#issuecomment-1535877560 > You are receiving this because your review was requested. > > Message ID: ***@***.***> -- Reply to this email directly or view it on GitHub: https://github.com/xrootd/xrootd/pull/2006#issuecomment-1535890965 You are receiving this because you are subscribed to this thread. Message ID: <[log in to unmask]> ######################################################################## Use REPLY-ALL to reply to list To unsubscribe from the XROOTD-DEV list, click the following link: https://listserv.slac.stanford.edu/cgi-bin/wa?SUBED1=XROOTD-DEV&A=1