<div dir="ltr">Hi Bron<div><br></div><div>Sorry for the delay and thank you for pushing this work to master.</div><div><br></div><div>A) While doing this, some of the logic was broken.  (This is now fix.)</div><div>B) The new solution works very well with a mix of archive (in object storage) and non archive (on regular spool) emails.</div><div>C) Unarchive have been fix.  (was used previously to remove email from object storage.  Now a new function is used for that).</div><div>D) Include reconstruct support.  Works very well if message.db file is reliable.<br></div><div><br></div><div>Please let me know if you can update master again from here <a href="https://github.com/RaymondPo/cyrus-imapd/tree/Cyrus-OpenIO">https://github.com/RaymondPo/cyrus-imapd/tree/Cyrus-OpenIO</a></div><div><br></div><div>When is the next meeting? Jean-Francois and I would like to attend</div><div><br></div><div>Regards</div><div>Raymond</div><div><br></div></div><div class="gmail_extra"><br><div class="gmail_quote">2015-10-13 8:03 GMT-04:00 Bron Gondwana <span dir="ltr"><<a href="mailto:brong@fastmail.fm" target="_blank">brong@fastmail.fm</a>></span>:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">I've pushed the objectstorage code based on Raymond and Jean-Francois's work to master.  It's almost exactly what was on Raymond's branch with just a couple of modifications.<br>
<br>
1) updated for some API changes I'd made to archiving in append.c<br>
2) fixed compilation under warn-unused-variables<br>
3) normalised logic between append_fromstage and append_copy for handling objectstorage errors by creating non-ARCHIVED files.<br>
4) fixed file copy in mailbox_archive to work the same way as the file moves, where it correctly passes the source/destination file to objectstorage and gets the file back, so unarchive works.<br>
<br>
WARNING WARNING WARNING: while I've tested that normal operation and normal archive file operation (non-objectstorage) works correctly with Cassandane and FastMail experimental branch, I haven't done more than rudimentary testing with the dummy objectstorage.<br>
<br>
I'm almost positive that reconstruct won't work correctly with object storage turned on right now.<br>
<br>
This is still very much work in progress - but putting it on master means that the code will be maintained and updated as other API changes are made.<br>
<br>
Thanks Raymond and Jean-Francois for all your work on this.  I hope it gets more testing and use.<br>
<br>
Cheers,<br>
<br>
Bron.<br>
<span class="HOEnZb"><font color="#888888"><br>
--<br>
  Bron Gondwana<br>
  <a href="mailto:brong@fastmail.fm">brong@fastmail.fm</a><br>
</font></span></blockquote></div><br></div>