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: | pgsql-committers(at)postgresql(dot)org |
Subject: | Re: pgsql-server/src/interfaces/libpgtcl pgtclCmds ... |
Date: | 2002-08-17 15:23:55 |
Message-ID: | 200208171523.g7HFNtH02253@candle.pha.pa.us |
Views: | Raw Message | Whole Thread | Download mbox | Resend email |
Thread: | |
Lists: | pgsql-committers |
No one reviewed it. Here is the email exchange I had with the author. I
specifically asked about the downside of the patch.
---------------------------------------------------------------------------
Tom Lane wrote:
> momjian(at)postgresql(dot)org (Bruce Momjian - CVS) writes:
> > Everytime if I do PQconsumeInput (when the backend channel gets
> > readable) I check for the return value. (0 == error) and generate a
> > notification manually, e.g. fixed string connection_closed) and pass it to the
> > TCL event queue.
>
> Was that patch reviewed by anyone? I don't think I've even seen it
> successfully posted --- the submitter seemed to be having trouble
> attaching it.
>
> >From the description, I don't think I trust it much. Event queue stuff
> is subtle ...
>
> > The only other thing I had to do is to comment out removing
> > all pending events in PgStopNotifyEventSource whenever the connection was
> > unexpectedly closed (so the manually generated event will not be deleted).
>
> ... and this is *very* likely to break things.
>
> regards, tom lane
>
> ---------------------------(end of broadcast)---------------------------
> TIP 2: you can get off all lists at once with the unregister command
> (send "unregister YourEmailAddressHere" to majordomo(at)postgresql(dot)org)
>
--
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
From | Date | Subject | |
---|---|---|---|
Next Message | Tom Lane | 2002-08-17 15:27:19 | Re: [COMMITTERS] pgsql-server/src backend/tcop/postgres.cbacke |
Previous Message | Tom Lane | 2002-08-17 15:14:57 | Re: pgsql-server/src/interfaces/libpgtcl pgtclCmds ... |