Log In - Home Page

Patch: Lock wait statistics

Edit Patch - Move To Another CommitFest - Delete Patch

CommitFest 2009-09
Topic Miscellaneous
Patch Status Returned with Feedback
Author Mark Kirkwood
Reviewers Jaime Casanova
Committer Nobody
Close Date 2009-10-05
Comments
Patch by markir on 2009-01-25 11:57:56 PM: Initial version
Comment by jcasanov on 2009-07-10 07:09:39 PM: Patch updated to apply in current HEAD.
Comment by rhaas on 2009-07-24 03:27:16 PM: Author is reworking patch, but Tom doubts the approach is viable.
Comment by tgl on 2009-07-24 05:08:09 PM: No, I said the proposed alternative didn't sound good. Not that I like this patch either; it's large, expensive, and presents a very ugly and non-future-proof API to users.
Comment by markir on 2009-07-26 06:13:11 PM: The API is one of the things I specifically mentioned as needed to be changed (see the original messages with the patch), I thought it should look similar to pg_locks. Not sure I can do much about the other objections (If I can see scope for less code I'll look into it!)
Comment by rhaas on 2009-08-02 03:14:47 PM: Moving to "Returned with Feedback".
Patch by markir on 2009-08-08 06:59:26 PM: New patch with max wait time added.
Patch by markir on 2009-08-08 07:05:30 PM: New patch with individual transaction lock waits merged.
Comment by sfrost on 2009-09-15 08:09:39 PM: ping'd on patch status and last 'todo' item.
Review by jcasanov on 2009-09-28 01:36:47 AM: we still have to get an agreement about what info we need to show
Review by rhaas on 2009-09-30 11:20:31 AM: Jeff Janes thinks this has some incorrect logic.
Comment by rhaas on 2009-10-05 07:16:38 PM: Moving to "Returned with Feedback".

Add Comment

Please log in to comment on this patch.