Log In - Home Page

Patch: add --throttle to pgbench

Edit Patch - Move To Another CommitFest - Delete Patch

CommitFest 2013-06
Topic Clients
Patch Status Committed
Author Fabien Coelho
Reviewers Greg Smith
Committer Tatsuo Ishii
Close Date 2013-07-22
Comments
Patch by calvin on 2013-05-02 08:16:08 AM: Initial submission, discussion about use cases in the thread.
Patch by calvin on 2013-05-02 12:13:39 PM: Second submission.
Patch by calvin on 2013-05-02 12:14:22 PM: Third submission and Greg Smith answer.
Patch by calvin on 2013-05-12 07:04:53 AM: Six. Minor update with help in its alphabetical position.
Patch by calvin on 2013-06-01 09:02:22 AM: Seven. Report transaction lag behind scheduled times.
Review by gsmith on 2013-06-08 11:54:25 PM: Review with simplified patch. Some weirdness around latency at the end of the test observed.
Patch by calvin on 2013-06-10 10:06:18 PM: v9 after Greg Smith review.
Patch by calvin on 2013-06-11 08:14:22 PM: v10 after Greg Smith remarks on v9.
Patch by calvin on 2013-06-15 07:58:35 PM: v11 to ensure that throttling is not included in latency measures.
Comment by maiku41 on 2013-06-21 08:59:05 PM: Changed to Waiting on Author based on reviewer feedback.
Comment by calvin on 2013-06-22 06:09:52 PM: The latency is due to PQfinish(), and is not related to the patch.
Patch by calvin on 2013-06-22 06:10:38 PM: v12 to interrupt clients which have not started a transaction
Patch by calvin on 2013-06-27 06:26:28 PM: v13 solves conflicts introduced by a recent commit.
Review by gsmith on 2013-07-14 05:56:48 PM: Review and a related pgbench finish refactoring.
Patch by gsmith on 2013-07-14 07:16:45 PM: v14 update with mainly comment changes
Patch by calvin on 2013-07-14 09:45:20 PM: v15 fixes comments in v14, no code change.
Patch by calvin on 2013-07-17 06:21:32 AM: v16 fixes conflicts introduced by a recent commit.
Patch by calvin on 2013-07-18 11:48:51 AM: v17 with updated & extended documentation about "lag", asked by Tatsuo.
Patch by gsmith on 2013-07-22 04:02:40 PM: v18 with more documentation updates

Add Comment

Please log in to comment on this patch.