BUG #7814: Rotation of the log is not carried out.

From: Jeff Janes <jeff(dot)janes(at)gmail(dot)com>
To: Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us>
Cc: Tsunezumi <tsunezumi(at)efficlabo(dot)com>, Kevin Grittner <kgrittn(at)mail(dot)com>, "pgsql-bugs(at)postgresql(dot)org" <pgsql-bugs(at)postgresql(dot)org>
Subject: BUG #7814: Rotation of the log is not carried out.
Date: 2013-01-24 02:26:15
Message-ID: CAMkU=1wqufGeocpC3=K9daiGAbR2Aw0zLRLX9cpPH_fov2huvw@mail.gmail.com
Views: Raw Message | Whole Thread | Download mbox | Resend email
Thread:
Lists: pgsql-bugs pgsql-hackers

On Tue, Jan 22, 2013 at 9:29 PM, Jeff Janes <jeff(dot)janes(at)gmail(dot)com> wrote:
> On Tuesday, January 22, 2013, Tom Lane wrote:
>>
>>
>> So what we need on Windows is for the data transfer thread to notice
>> when "Log_RotationSize > 0 && ftell(syslogFile) >= Log_RotationSize",
>> and then either signal the control thread to wake up, or do the
>> rotation itself. Probably the former is less risk.
>>
>> I'm not coding this fix though, since I'm not in a position to test it.
>
>
> If I can follow though on Andrew Dunstan's instructions to get mingw up
and
> running, I'll take it for a spin.

The attached patch fixes the problem, tested under mingw.

I've included an elog "JJ logger woke up" in the patch to verify that the
wake up is not happening more often than it should. Obviously this line
should not be included in the actual commit--but I thought a tester would
like to see how I verified this.

This is my first effort in both Windows and in the Latch code, so committer
beware.

It should be applied to 9_2 and to head.

I've not added it to the commit-fest as it is a bug fix.

Cheers,

Jeff

Attachment Content-Type Size
syslogger_rotate_win_v1.patch application/octet-stream 828 bytes

In response to

Responses

Browse pgsql-bugs by date

  From Date Subject
Next Message Jeff Janes 2013-01-24 03:06:13 Re: Re: BUG #7748: "drop owned by" fails with error message: "unrecognized object class: 1262"
Previous Message Pius Chan 2013-01-23 23:23:50 Re: BUG #7819: missing chunk number 0 for toast value 1235919 in pg_toast_35328

Browse pgsql-hackers by date

  From Date Subject
Next Message Craig Ringer 2013-01-24 03:03:55 Re: Visual Studio 2012 RC
Previous Message Noah Misch 2013-01-24 02:12:05 Re: patch submission: truncate trailing nulls from heap rows to reduce the size of the null bitmap [Review]