This is the same as the not yet committed core patch at #802856: Make lock_wait() wait less - changes lock_wait() to poll every 10th of a second instead of every second.

CommentFileSizeAuthor
#6 969998_lock_wait.patch2.22 KBcatch
#1 lock_wait_less.patch1.12 KBcatch

Comments

catch’s picture

catch
he/him
Primary language English
commented
Title: Shorter lock_wait() time » Shorter lock_wait() polling time
Status: Active » Needs review
StatusFileSize
new lock_wait_less.patch1.12 KB
douggreen’s picture

douggreen commented

This is a great idea!

I think we also need to decrease the lock_wait time because a 30 second delay on any web process is excessive. If we can't change the default, then there are probably a few places which should pass a lower default.

catch’s picture

catch
he/him
Primary language English
commented
Status: Needs review » Needs work

There's a better patch at #802856: Make lock_wait() wait less now which I need to port across to memcache. CNW for now.

catch’s picture

catch
he/him
Primary language English
commented
Assigned: Unassigned » catch
catch’s picture

catch
he/him
Primary language English
commented
Version: 7.x-1.x-dev » 6.x-1.x-dev
Status: Needs work » Needs review

Updated to mirror the (still pending after several months) core patch at #802856: Make lock_wait() wait less.

Explanation for that patch is at http://drupal.org/node/802856#comment-4015000 to give more background to this issue.

Posting 6.x version but this should be trivial forward-port to 7.x.

catch’s picture

catch
he/him
Primary language English
commented
StatusFileSize
new 969998_lock_wait.patch2.22 KB
catch’s picture

catch
he/him
Primary language English
commented
Version: 6.x-1.x-dev » 7.x-1.x-dev

Confirmed that lock tests continue to pass with this applied, committed to 6.x. Needs to go into 7.x too.

jeremy’s picture

jeremy commented
Status: Needs review » Patch (to be ported)

Marking as to-be-ported, as my understanding is that the 6.x patch now needs to be ported to 7.x.

catch’s picture

catch
he/him
Primary language English
commented
Status: Patch (to be ported) » Fixed

Committed. http://drupalcode.org/project/memcache.git/commit/2fe64c5

Status: Fixed » Closed (fixed)

Automatically closed -- issue fixed for 2 weeks with no activity.