Re: droped out precise time calculations in src/interfaces/libpq/fe-connect.c

From: Bruce Momjian <pgman(at)candle(dot)pha(dot)pa(dot)us>
To: Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us>
Cc: PostgreSQL-development <pgsql-hackers(at)postgresql(dot)org>, Joe Conway <mail(at)joeconway(dot)com>, Denis A Ustimenko <denis(at)oldham(dot)ru>
Subject: Re: droped out precise time calculations in src/interfaces/libpq/fe-connect.c
Date: 2002-10-14 22:41:27
Message-ID: 200210142241.g9EMfRY27876@candle.pha.pa.us
Views: Raw Message | Whole Thread | Download mbox | Resend email
Thread:
Lists: pgsql-hackers

Tom Lane wrote:
> Bruce Momjian <pgman(at)candle(dot)pha(dot)pa(dot)us> writes:
> > /*
> > * select() may modify timeout argument on some platforms so
> > ! * use copy.
> > ! * XXX Do we really want to do that? If select() returns
> > ! * the number of seconds remaining, we are resetting
> > ! * the timeout to its original value. This will yeild
> > ! * incorrect timings when select() is interrupted.
> > ! * bjm 2002-10-14
> > */
> > tmp_timeout = *timeout;
> > ptmp_timeout = &tmp_timeout;
>
>
> Actually, now that I look at this, the API for PQwaitTimed is wrong
> after all. The right way to implement this is for the caller to pass in
> finish_time (or some indication that no timeout is wanted). Evaluation
> of the time left to wait should happen inside this retry loop. That
> way, you get the right behavior (plus or minus one second, anyway)
> independently of whether the platform's select() reduces its timeout
> argument or not.

Yes, you are saying do the time() inside PQwaitTimed(), so we can
properly get new time() values on select() retry. Yep.

--
Bruce Momjian | http://candle.pha.pa.us
pgman(at)candle(dot)pha(dot)pa(dot)us | (610) 359-1001
+ If your life is a hard drive, | 13 Roberts Road
+ Christ can be your backup. | Newtown Square, Pennsylvania 19073

In response to

Browse pgsql-hackers by date

  From Date Subject
Next Message Bruce Momjian 2002-10-14 22:53:27 Re: orderRules() now a bad idea?
Previous Message am 2002-10-14 21:13:58 Re: Anyone want to assist with the translation of the Advocacy site?