Bump up PG_CONTROL_VERSION on HEAD

Started by Michael Paquierabout 7 years ago3 messageshackers
Jump to latest
#1Michael Paquier
michael@paquier.xyz

Hi all,

f3db7f16 has proved that it can be a bad idea to run pg_resetwal on a
data folder which does not match the version it has been compiled
with.

As of HEAD, PG_CONTROL_VERSION is still 1100:
$ pg_controldata | grep "pg_control version"
pg_control version number: 1100

Wouldn't it be better to bump it up to 1200?

Thanks,
--
Michael

#2Andres Freund
andres@anarazel.de
In reply to: Michael Paquier (#1)
Re: Bump up PG_CONTROL_VERSION on HEAD

Hi,

On 2019-01-16 11:02:08 +0900, Michael Paquier wrote:

f3db7f16 has proved that it can be a bad idea to run pg_resetwal on a
data folder which does not match the version it has been compiled
with.

As of HEAD, PG_CONTROL_VERSION is still 1100:
$ pg_controldata | grep "pg_control version"
pg_control version number: 1100

Wouldn't it be better to bump it up to 1200?

We don't commonly bump that without corresponding control version
changes. I don't see what we'd gain by the bump?

Greetings,

Andres Freund

#3Tom Lane
tgl@sss.pgh.pa.us
In reply to: Andres Freund (#2)
Re: Bump up PG_CONTROL_VERSION on HEAD

Andres Freund <andres@anarazel.de> writes:

On 2019-01-16 11:02:08 +0900, Michael Paquier wrote:

f3db7f16 has proved that it can be a bad idea to run pg_resetwal on a
data folder which does not match the version it has been compiled
with.

As of HEAD, PG_CONTROL_VERSION is still 1100:
$ pg_controldata | grep "pg_control version"
pg_control version number: 1100

Wouldn't it be better to bump it up to 1200?

We don't commonly bump that without corresponding control version
changes. I don't see what we'd gain by the bump?

Yeah, it has not been our practice to bump PG_CONTROL_VERSION
unless the contents of pg_control actually change. The whole
point of f3db7f16 was to ensure that we didn't have to do that
just because of a major version change.

regards, tom lane