On Fri, 2012-09-14 at 17:58 -0700, Jeff Davis wrote:
> This is just a rebased version of the patch by Simon here:

I just noticed the following note in the docs for this patch:

  The default is <literal>off</> for backwards compatibility and
  to allow upgrade. The recommended setting is <literal>on</> though
  this should not be enabled until upgrade is successfully complete
  with full set of new backups.

I don't understand what that means -- if they have the page_checksums
GUC available, then surely upgrade is complete, right? And what is the
backwards-compatibility issue?

Also, it looks out of date, because the default in guc.c is set to true.
I think we should probably default to true, because it's safer and it
can always be disabled at runtime, but I don't have a strong opinion
about that.

Regards,
	Jeff Davis

In response to

• WIP checksums patch at 2012-09-15 00:58:37 from Jeff Davis

Responses

• Re: WIP checksums patch at 2012-10-01 14:43:00 from Bruce Momjian

pgsql-hackers by date

Next:	From: Jeff Davis	Date: 2012-10-01 02:19:04
	Subject: Re: WIP checksums patch
Previous:	From: Jeff Davis	Date: 2012-10-01 01:15:06
	Subject: Re: gistchoose vs. bloat