Re: Checksum errors in pg_stat_database

From: Robert Treat <rob(at)xzilla(dot)net>
To: Julien Rouhaud <rjuju123(at)gmail(dot)com>
Cc: Magnus Hagander <magnus(at)hagander(dot)net>, Michael Paquier <michael(at)paquier(dot)xyz>, David Steele <david(at)pgmasters(dot)net>, Tomas Vondra <tomas(dot)vondra(at)2ndquadrant(dot)com>, Robert Haas <robertmhaas(at)gmail(dot)com>, PostgreSQL-development <pgsql-hackers(at)postgresql(dot)org>
Subject: Re: Checksum errors in pg_stat_database
Date: 2019-04-17 13:51:25
Message-ID: CAJSLCQ0HUMWN_zsD85kH=11pSFr6kAwj3v1-5FFJ-O+wf7DjNg@mail.gmail.com
Views: Raw Message | Whole Thread | Download mbox | Resend email
Thread:
Lists: pgsql-hackers

On Wed, Apr 17, 2019 at 9:07 AM Julien Rouhaud <rjuju123(at)gmail(dot)com> wrote:
>
> On Wed, Apr 17, 2019 at 1:55 PM Magnus Hagander <magnus(at)hagander(dot)net> wrote:
> >
> > On Tue, Apr 16, 2019 at 5:39 PM Robert Treat <rob(at)xzilla(dot)net> wrote:
> >>
> >> On Mon, Apr 15, 2019 at 3:32 PM Julien Rouhaud <rjuju123(at)gmail(dot)com> wrote:
> >> >
> >> > I don't know if that counts as an open item, but I attach a patch for
> >> > all points discussed here.
> >>
> >> ISTM we should mention shared objects in both places in the docs, and
> >> want "NULL if data checksums" rather than "NULL is data checksums".
> >> Attaching slightly modified patch with those changes, but otherwise
> >> LGTM.
>
> Thanks, that's indeed embarassing typos. And agreed for mentioning
> shared objects in both places.
>
> >
> > Interestingly enough, that patch comes out as corrupt. I have no idea why though :) v1 is fine.
> >
> > So I tried merging back your changes into it, and then pushing. Please doublecheck I didn't miss something :)
>
> Thanks! I double checked and it all looks fine.

+1

Robert Treat
https://xzilla.net

In response to

Browse pgsql-hackers by date

  From Date Subject
Next Message Justin Pryzby 2019-04-17 14:01:15 Re: ToDo: show size of partitioned table
Previous Message Masahiko Sawada 2019-04-17 13:38:11 Re: New vacuum option to do only freezing