add periodicFlush option

pinojs / sonic-boom

Extremely fast utf8 only stream implementation

MIT License

266 stars 41 forks source link

add periodicFlush option #200

Closed forgotPassword closed 2 months ago

forgotPassword commented 3 months ago

fixes #193

forgotPassword commented 3 months ago

added suggested changes

forgotPassword commented 3 months ago

should _periodicFlush be periodicFlush? but then there is _fsync. also some tests (before my changes) are flaky on wsl, don't know if it's a problem

 FAIL  test/write.test.js 4 failed of 120 19s
 ✖ sync true > write things to a file descriptor > EBADF: bad file descriptor, close
    code: EBADF
    errno: -9
    syscall: close
    tapCaught: uncaughtException