make lock on republisher-checkin stronger

Bug #871823 reported by danh
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Scribe2
New
Undecided
danh

Bug Description

It looks like in some rare cases, the lock on republisher-checkin is not strong enough to present races that start if two checkin processes are launched very near each other.

I think this could be fixed by the methods used in Bug #684492 for the scandata, or perhaps in this case we can do something simpler (like just checking for the existence of the file immediately before writing it ---- so we'd check existence twice: once at the start, and once when we start writing the lock file).

Revision history for this message
danh (danh-archive) wrote :

I should have noted that Paul referred me to a specimen in the wild that Hank caught, klinischestasche00corn from scribe3 in boston, uploaded 10/05/2011.

Changed in scribe2:
assignee: nobody → danh (danh-archive)
Revision history for this message
danh (danh-archive) wrote :

We added a second check for the existence of the lock file immediately before creating it (there was already a check, but much earlier). This is in revision 39141 (RePublisher-checkin.php).

To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.