Bruce Momjian wrote: > Patch applied. I am applying it so it is in CVS and everyone can see > it. I will keep modifying it until everyone likes it. It is just > easier to do it that way when multiple people are reviewing it. They > can jump in and make changes too.
I ran the same test as before, with the following results: current cvs ----------- good connect info, using hostaddr, timeout = 1 second ===================================================== unsuccessful 0 times: avg n/a successful 50000 times: avg 0.0087 bad connect info, using hostaddr, timeout = 1 second ==================================================== unsuccessful 100 times: avg 1.4998 successful 0 times: sum 0: avg n/a Seems to work well. But one slight concern: with previous 2 line patch -------------------------- good connect info, using hostaddr, timeout = 1 || 2 second(s) ============================================================= unsuccessful 0 times: avg n/a successful 20000 times: avg 0.0074 These tests were on the same, otherwise unloaded development box. Not sure if it is an artifact or not, but the average connection time has gone from 0.0074 to 0.0087, an increase of about 17%. Also worth noting is that there was very little deviation from connect-to-connect on both of the tests (eye-balled the total range at about 0.0003). I did not bother calculating standard deviation of the connect times, but I'm certain it would not be enough to account for the difference. Could anything in Bruce's patch account for this, or do you think it is normal variation due to something on my dev box? Joe ---------------------------(end of broadcast)--------------------------- TIP 4: Don't 'kill -9' the postmaster