mafintosh: will do. Sadly Im now finding that if I run the tests without debug logging on, I still get failures, which makes me think there's a timing issue that's affected by the logging
mafintosh
well that seems like a great fix no matter what
pfrazee
yeah definitely
Im not seeing signs of that bug, or the RAF issue, now - so we may have a THIRD bug at play. Dun dun dunnn
dat-gitter-bot1 has quit
dat-gitter-bot joined the channel
mafintosh: Im looking forward to the day we get to reimplement the dat stack in rust, and then compile it into wasm and/or native
mafintosh
yea
agreed
pfrazee: i'm gonna redeploy the archiver with your fix
toddself has quit
pfrazee
mafintosh: ok. Turns out there's still cases where the RAF is closed when the write is attempted, so I should have another patch on the way
mafintosh
pfrazee: thats fixes the remaining bug?
pfrazee
mafintosh: it's the same failing test - sometimes sync doesnt complete
mafintosh: my patch fixed one case that caused that, but there still appears to be another cause
mafintosh: is it possible that RAF instances are shared between the meta and content hypercores of an archive?
johncoffee joined the channel
Im trying to wrap my head around why the RAF instance is getting closed prematurely