Peter Brant wrote: > Ah, sorry for the late response (and for any confusion), but the only > thing I tested was Qingqing's rewritten semaphore implementation. I > didn't test the proposed bug fixes to the existing semaphore > implementation.
Advertising
Oh, OK. The email mentioned the semaphore patch without specifying which one, so I assume it was the shorter one. > We've never been able reproduce (or even trigger) the original "sem_ctl > failed" error in a testing environment so it would be hard to say if the > changes to win32/sema.c have an impact on it or not. On the other hand, > win32_sema.c seems to solve the pgbench lockups reported earlier by Jim > N. and it successfully completes a reasonably brutal stress test with > real world data and real world queries (which at least is a good > indication that it basically works). OK, let's consider the item closed. We didn't backpatch the new win32_sema.c file to 8.1.X or 8.0.X, so let'see if we get more reports. Thanks. -- Bruce Momjian http://candle.pha.pa.us EnterpriseDB http://www.enterprisedb.com + If your life is a hard drive, Christ can be your backup. + ---------------------------(end of broadcast)--------------------------- TIP 6: explain analyze is your friend
