Re: [HACKERS] Infrastructure changes for recovery

On Fri, 2008-09-12 at 14:14 -0400, Alvaro Herrera wrote:
> Simon Riggs wrote:
>
> > --- 5716,5725 ----
> >                           CheckpointStats.ckpt_sync_end_t,
> >                           &sync_secs, &sync_usecs);
> >
> > !     elog(LOG, "%s complete: wrote %d buffers (%.1f%%); "
> >            "%d transaction log file(s) added, %d removed, %d recycled; "
> >            "write=%ld.%03d s, sync=%ld.%03d s, total=%ld.%03d s",
> > +          (checkpoint ? "  checkpoint" : "restartpoint"),
> >            CheckpointStats.ckpt_bufs_written,
> >            (double) CheckpointStats.ckpt_bufs_written * 100 / NBuffers,
> >            CheckpointStats.ckpt_segs_added,
>
> Very minor nit: this really needs a rework.

All I changed was the word "restartpoint"... its otherwise identical to
existing message. I'd rather not change that.

--
 Simon Riggs           www.2ndQuadrant.com
 PostgreSQL Training, Services and Support