Re: [Review] pgbench duration option - Mailing list pgsql-hackers

From Heikki Linnakangas
Subject Re: [Review] pgbench duration option
Date
Msg-id 48C8EEFB.1040901@enterprisedb.com
Whole thread Raw
In response to Re: [Review] pgbench duration option  (ITAGAKI Takahiro <itagaki.takahiro@oss.ntt.co.jp>)
Responses Re: [Review] pgbench duration option  (Magnus Hagander <magnus@hagander.net>)
List pgsql-hackers
ITAGAKI Takahiro wrote:
> Here is a revised version of the pgbench duration patch.

Looking at the Win32 timer implementation, it's a bit different from the 
one we have in src/backend/port/win32/timer.c. The one in timer.c uses a 
separate thread and WaitForSingleObjectEx() to wait, while your 
implementation uses CreateTimerQueue() and CreateTimerQueueTimer(). 
Yours seems simpler, so I wonder why the timer.c is different?

It's not too bad as it is in the patch, but it would be nice to put the 
setitimer() implementation into src/port, and use the same code in the 
backend as well.

--   Heikki Linnakangas  EnterpriseDB   http://www.enterprisedb.com


pgsql-hackers by date:

Previous
From: Richard Huxton
Date:
Subject: Re: Transaction Snapshots and Hot Standby
Next
From: Andrew Dunstan
Date:
Subject: Re: Postgresql coding conventions