NHacker Next
  • new
  • past
  • show
  • ask
  • show
  • jobs
  • submit
A Syncthing and SQLite Gotcha (borretti.me)
zbentley 1 days ago [-]
Yes, handle statefulness is very important.

A cheap way to detect and handle this condition in your application code is to call fstat on the open file handle and check the number of references. If that number drops to zero, you have an orphan. Using fstat on the handle is cheap enough that you can affordably do it on every query even at high traffic.

I expanded on this approach in a previous comment on a project which live-syncs SQLite changes (which might be useful for your application) here: https://news.ycombinator.com/item?id=47884461

boringstack 1 days ago [-]
the fstat reference-count check is a neat trick - cheap enough to run per-query changes the calculus from 'detect after the incident' to 'assert continuously'. adding it to the deploy checklist next to the restore drill.
zbentley 1 days ago [-]
Eh, I think you can add it to the application’s write path everywhere and not need it at deployment time.

Edit: I had vouched for and replied to parent, but it appears to be an AI HN responder that substantially regurgitates summaries of comments. That is a violation of the rules as I understand them. Un-vouched.

zbentley 1 days ago [-]
I think there is an important typo:

> what happens if other processes had open file descriptors pointing to new

Should end with “old”, not “new”.

zetalyrae 24 hours ago [-]
Ah, no, because how you'd use rename in this context is something like:

    rename("/tmp/foobar", "/path/to/real/foobar");
So, `new` is really `destination`.
euroderf 1 days ago [-]
Shouldn't this return errors to the "other processes [that] had open file descriptors pointing to new" ?
boringstack 1 days ago [-]
[flagged]
Guidelines | FAQ | Lists | API | Security | Legal | Apply to YC | Contact
Rendered at 23:21:35 GMT+0000 (Coordinated Universal Time) with Vercel.