From: | Bruce Momjian <pgman(at)candle(dot)pha(dot)pa(dot)us> |
---|---|
To: | Magnus Hagander <mha(at)sollentuna(dot)net> |
Cc: | Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us>, PostgreSQL-patches <pgsql-patches(at)postgresql(dot)org>, Andreas Pflug <pgadmin(at)pse-consulting(dot)de> |
Subject: | Re: [BUGS] BUG #1466: #maintenace_work_mem = 16384 |
Date: | 2005-02-27 01:03:34 |
Message-ID: | 200502270103.j1R13ZT24350@candle.pha.pa.us |
Views: | Raw Message | Whole Thread | Download mbox | Resend email |
Thread: | |
Lists: | pgsql-patches |
Patch applied. Thanks.
I assume this is not for 8.0.X.
---------------------------------------------------------------------------
Magnus Hagander wrote:
> >>> The proposed test on Redirect_stderr looks pretty fishy too; for one
> >>> thing it will almost certainly not be the right thing
> >inside the stderr
> >>> logger subprocess itself.
> >
> >> Could you explain further what the issue is there?
> >
> >Inside the logger subprocess, Redirect_stderr is guaranteed true (since
> >it'll be inherited from the postmaster) and therefore the proposed
> >change ensures that anything the logger might want to complain about
> >goes to the original stderr, ie, into the bit bucket rather than
> >someplace useful. Perhaps something like
> >
> > if ((!Redirect_stderr || am_syslogger) && pgwin32_is_service())
> >
> >would be reasonable.
>
> <snip lots of others>
>
> Here is an updated patch, that should take care of this. Tested that it
> solves the problem reported.
>
>
> >> There is special code in the send_message_to_server_log
> >function to make
> >> sure it's written directly to the file.
> >
> >If the logger is complaining, it's quite possibly because it's
> >unable to
> >write to its file. Now that you mention it, doesn't this code go into
> >infinite recursion if write_syslogger_file_binary() tries to ereport?
> >
>
> I haven't looked at this part, it appears a separate (but closely
> related) issue.
>
> Perhaps Andreas can comment on this?
>
> //Magnus
Content-Description: stderr.patch
[ Attachment, skipping... ]
>
> ---------------------------(end of broadcast)---------------------------
> TIP 5: Have you checked our extensive FAQ?
>
> http://www.postgresql.org/docs/faq
--
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 | Bruce Momjian | 2005-02-27 01:07:22 | Re: [HACKERS] UTF8 or Unicode |
Previous Message | Bruce Momjian | 2005-02-27 00:55:40 | Re: [HACKERS] UTF8 or Unicode |