Re: WIP XLog Switch

Lists: pgsql-patches
From: Simon Riggs <simon(at)2ndquadrant(dot)com>
To: pgsql-patches(at)postgresql(dot)org
Subject: WIP XLog Switch
Date: 2005-04-19 18:44:36
Message-ID: 1113936276.16721.2174.camel@localhost.localdomain
Views: Raw Message | Whole Thread | Download mbox | Resend email
Lists: pgsql-patches


WIP patch to perform a switch from one log file to next when we do a
pg_stop_backup(), including wal replay handling at recovery time.

Patch currently crashes server at various points, so don't stare too
hard, but patch applies cleanly on cvstip, compiles and make checks.

Main issue is the need to poke the xlog record pointer with a new value
after the log switch. I'm a little uncertain about that approach and I'm
very likely getting it wrong now. Better ideas welcome.

Patch is incomplete in that it doesn't handle shutdown checkpoints as
log switches in archive mode (yet)

Also nothing in here about standby databases (yet)

Any comments appreciated before I spend too much time on this.

Best Regards, Simon Riggs

Attachment Content-Type Size
logswitch1.patch text/x-patch 16.5 KB

From: Simon Riggs <simon(at)2ndquadrant(dot)com>
To: pgsql-patches(at)postgresql(dot)org
Subject: Re: WIP XLog Switch
Date: 2005-04-20 08:27:51
Message-ID: 1113985671.16721.2194.camel@localhost.localdomain
Views: Raw Message | Whole Thread | Download mbox | Resend email
Lists: pgsql-patches

On Tue, 2005-04-19 at 19:44 +0100, Simon Riggs wrote:
> Main issue is the need to poke the xlog record pointer with a new value
> after the log switch. I'm a little uncertain about that approach and I'm
> very likely getting it wrong now. Better ideas welcome.

Happier with that approach now. Working on other issues in code.

Best Regards, Simon Riggs


From: Bruce Momjian <pgman(at)candle(dot)pha(dot)pa(dot)us>
To: Simon Riggs <simon(at)2ndquadrant(dot)com>
Cc: pgsql-patches(at)postgresql(dot)org
Subject: Re: WIP XLog Switch
Date: 2005-05-21 03:16:40
Message-ID: 200505210316.j4L3GeJ04990@candle.pha.pa.us
Views: Raw Message | Whole Thread | Download mbox | Resend email
Lists: pgsql-patches


Any farther on this?

---------------------------------------------------------------------------

Simon Riggs wrote:
>
> WIP patch to perform a switch from one log file to next when we do a
> pg_stop_backup(), including wal replay handling at recovery time.
>
> Patch currently crashes server at various points, so don't stare too
> hard, but patch applies cleanly on cvstip, compiles and make checks.
>
> Main issue is the need to poke the xlog record pointer with a new value
> after the log switch. I'm a little uncertain about that approach and I'm
> very likely getting it wrong now. Better ideas welcome.
>
> Patch is incomplete in that it doesn't handle shutdown checkpoints as
> log switches in archive mode (yet)
>
> Also nothing in here about standby databases (yet)
>
> Any comments appreciated before I spend too much time on this.
>
> Best Regards, Simon Riggs

[ Attachment, skipping... ]

>
> ---------------------------(end of broadcast)---------------------------
> TIP 3: if posting/reading through Usenet, please send an appropriate
> subscribe-nomail command to majordomo(at)postgresql(dot)org so that your
> message can get through to the mailing list cleanly

--
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: Simon Riggs <simon(at)2ndquadrant(dot)com>
To: Bruce Momjian <pgman(at)candle(dot)pha(dot)pa(dot)us>
Cc: pgsql-patches(at)postgresql(dot)org
Subject: Re: WIP XLog Switch
Date: 2005-05-21 17:38:02
Message-ID: 1116697082.3844.352.camel@localhost.localdomain
Views: Raw Message | Whole Thread | Download mbox | Resend email
Lists: pgsql-patches

On Fri, 2005-05-20 at 23:16 -0400, Bruce Momjian wrote:
> Any farther on this?

No, but it will be in by deadline.

I need to make first base with partitioning before I can do this. Hence
my (otherwise rude) ignoring of your sterling work on the exec logging
patch.

Best Regards, Simon Riggs


From: Simon Riggs <simon(at)2ndquadrant(dot)com>
To: Bruce Momjian <pgman(at)candle(dot)pha(dot)pa(dot)us>
Cc: pgsql-patches(at)postgresql(dot)org
Subject: Re: WIP XLog Switch
Date: 2005-07-19 23:20:57
Message-ID: 1121815257.3998.47.camel@localhost.localdomain
Views: Raw Message | Whole Thread | Download mbox | Resend email
Lists: pgsql-patches

On Sat, 2005-05-21 at 18:38 +0100, Simon Riggs wrote:
> On Fri, 2005-05-20 at 23:16 -0400, Bruce Momjian wrote:
> > Any farther on this?
>
> No, but it will be in by deadline.
>
> I need to make first base with partitioning before I can do this. Hence
> my (otherwise rude) ignoring of your sterling work on the exec logging
> patch.

If I were to complete this patch now over next few days, would anybody
object to its inclusion in 8.1 beta?

I don't like to leave things incomplete...

Best Regards, Simon Riggs


From: Bruce Momjian <pgman(at)candle(dot)pha(dot)pa(dot)us>
To: Simon Riggs <simon(at)2ndquadrant(dot)com>
Cc: pgsql-patches(at)postgresql(dot)org
Subject: Re: WIP XLog Switch
Date: 2005-07-29 15:22:24
Message-ID: 200507291522.j6TFMOC12595@candle.pha.pa.us
Views: Raw Message | Whole Thread | Download mbox | Resend email
Lists: pgsql-patches

Simon Riggs wrote:
> On Sat, 2005-05-21 at 18:38 +0100, Simon Riggs wrote:
> > On Fri, 2005-05-20 at 23:16 -0400, Bruce Momjian wrote:
> > > Any farther on this?
> >
> > No, but it will be in by deadline.
> >
> > I need to make first base with partitioning before I can do this. Hence
> > my (otherwise rude) ignoring of your sterling work on the exec logging
> > patch.
>
> If I were to complete this patch now over next few days, would anybody
> object to its inclusion in 8.1 beta?
>
> I don't like to leave things incomplete...

Please send us what you have when you are done and we will review it.

--
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