Log In - Home Page

Patch: Timeout framework extension and lock_timeout (from 2012-11)

Edit Patch - Move To Another CommitFest - Delete Patch

CommitFest 2013-01
Topic Server Features
Patch Status Committed
Author Zoltán Böszörményi
Reviewers Tom Lane, Hari Babu, Stephen Frost
Committer Tom Lane
Close Date 2013-03-17
Comments
Patch by zboszor on 2012-07-22 08:07:03 PM: Initial version.
Patch by zboszor on 2012-08-21 02:32:44 PM: New, cleaned up version. The new timeouts are called Lock Manager Timeouts, deadlock_timeout is converted to this scheme.
Review by tgl on 2012-09-22 06:50:33 PM: Needs a lot of work yet.
Patch by zboszor on 2012-09-24 09:33:16 PM: Didn't need so much work, only a day. All points were addressed.
Patch by zboszor on 2012-10-03 09:43:10 AM: Bugfix and tweaking comments.
Comment by alvherre on 2012-10-23 02:00:12 PM: moved from CF2
Comment by ringerc on 2013-01-18 02:44:15 AM: Stale, needs a re-test before commit
Comment by ringerc on 2013-01-18 02:49:01 AM: Requested follow-up from author with current test state, Windows tests
Review by ringerc on 2013-01-29 12:02:53 AM: Tested. Needs rebase, regress .out file not updated correctly. Attached document is tests run on patch during eval.
Patch by zboszor on 2013-01-30 02:43:57 PM: Rebased patch.
Patch by zboszor on 2013-01-31 01:56:52 PM: Update regression test output for the max_prepared_transactions=0 case.
Review by sfrost on 2013-02-23 01:58:54 AM: Another review, would like feedback from the author.
Patch by zboszor on 2013-02-24 12:32:20 PM: List based multiple timeouts API, separate per-lock and per-stmt variants, modified comments and doc.
Patch by zboszor on 2013-02-28 08:24:43 AM: R.I.P. per-statement lock_timeout. (May be submitted later.)
Comment by ringerc on 2013-03-04 03:25:58 AM: Note that the rest of the patch is still in flight, it's only one part of it that's been dropped for now.
Review by tgl on 2013-03-15 05:54:43 PM: Don't like some of the API changes, am worried about race conditions introduced into lock management
Patch by zboszor on 2013-03-16 04:03:54 PM: New version, hopefully without the race condition and with the other requested changes.

Add Comment

Please log in to comment on this patch.