<div dir="ltr">I did indeed have the wrong idea from only having skimmed that thread, thanks for clarifying.  I'm glad to know flush wasn't on the chopping block afterall.  :)<div><br></div><div> - Chris</div></div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Mon, Jan 13, 2020 at 4:44 PM Phil B <<a href="mailto:pbpublist@gmail.com" target="_blank">pbpublist@gmail.com</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex"><div dir="ltr">Chris,<div><br></div><div>FWIW, if it's the thread I think it is, you got the wrong impression from the discussion.  The assertion was that the flush/close/reopen dance on the changes file was no longer needed, *not* that flush was no longer necessary.  The flush was the part that we kept! (i.e. the argument was that it's no longer the 90's and that we should trust flush to behave as expected)</div><div><br></div><div>Thanks,</div><div>Phil</div></div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Sun, Jan 12, 2020 at 6:18 PM Chris Muller <<a href="mailto:asqueaker@gmail.com" target="_blank">asqueaker@gmail.com</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex"><div dir="ltr">Magma has been stable in 5.2 for a long time under an older VM, all tests pass.  But by changing ONLY the VM (not the image) to the new release-candidate, it fails the forward-recovery test.  This test tests the scenario of a server failure during mid-write.  Unless I change StandardFileStream>>#flush as in Files-cmm.182, the recovery data which Magma relies on #flush to ensure is preserved is, in fact, not preserved.  It appears to be a breakage of the contract which causes the test to fail.  This functionality is important to avoid corrupting databases.<div><div><br></div><div>I saw a discussion on the Cuis list in which someone was asserting that flush is no longer necessary(!!), and made a vague reference to a "thread on squeak-dev" which I never found.</div><div><br></div><div>I hope this is just an oversight, otherwise I'll have to rely something like Files-cmm.182, which is half the speed of the old #flush.</div></div><div><br></div><div>Best,</div><div>  Chris</div></div>
<br>
</blockquote></div>
<br>
</blockquote></div>