]> Sergey Matveev's repositories - public-inbox.git/commit
imaptracker: use flock(2) around writes
authorEric Wong <e@yhbt.net>
Sat, 27 Jun 2020 10:03:59 +0000 (10:03 +0000)
committerEric Wong <e@yhbt.net>
Sun, 28 Jun 2020 22:29:39 +0000 (22:29 +0000)
commit35e31227139e401cf7845d824a1b4e6a1de56fe3
tree78bc53a4e5cfe47486e6a7882a078e73836dc051
parent6968c91707d51f6a60d476cc2f5d65a660146813
imaptracker: use flock(2) around writes

SQLite only issues non-blocking F_SETLK ops (not F_SETLKW) and
retries failures using a configurable busy_timeout.  SQLite's
busy loop sleeps for a millisecond and retries the lock until
the configured busy_timeout is hit.

Trying to set ->sqlite_busy_timeout to larger values (e.g. 30000
milliseconds) still leads to failure when running the new stress
test with 8 processes with TMPDIR on a 7200 RPM HDD.

Inspection of SQLite source reveals there's no built-in way to
use F_SETLKW, so tack on the existing flock(2) support we use to
synchronize git + SQLite + Xapian for inbox writing.  We use
flock(2) instead of POSIX fcntl(2) locks since Perl doesn't
provide a way to manipulate "struct flock" portably.
lib/PublicInbox/IMAPTracker.pm
t/imap_tracker.t