pgsql: Add warning to pg_controldata on PG_CONTROL_VERSION mismatch
| От | Heikki Linnakangas |
|---|---|
| Тема | pgsql: Add warning to pg_controldata on PG_CONTROL_VERSION mismatch |
| Дата | |
| Msg-id | E1vIrqz-0066AU-19@gemulon.postgresql.org обсуждение исходный текст |
| Список | pgsql-committers |
Add warning to pg_controldata on PG_CONTROL_VERSION mismatch If you run pg_controldata on a cluster that has been initialized with different PG_CONTROL_VERSION than what the pg_controldata program has been compiled with, pg_controldata will still try to interpret the control file, but the result is likely to be somewhat nonsensical. How nonsensical it is depends on the differences between the versions. If sizeof(ControlFileData) differs between the versions, the CRC will not match and you get a warning of that, but otherwise you get no warning. Looking back at recent PG_CONTROL_VERSION updates, all changes that would mess up the printed values have also changed sizeof(ControlFileData), but there's no guarantee of that in future versions. Add an explicit check and warning for version number mismatch before the CRC check. That way, you get a more clear warning if you use the pg_controldata binary from wrong version, and if we change the control file in the future in a way that doesn't change sizeof(ControlFileData), this ensures that you get a warning in that case too. Discussion: https://www.postgresql.org/message-id/2afded89-f9f0-4191-84d8-8b8668e029a1@iki.fi Branch ------ master Details ------- https://git.postgresql.org/pg/commitdiff/6956bca515e2a07a16c6d68b208766e97b275ddc Modified Files -------------- src/bin/pg_controldata/pg_controldata.c | 9 ++++++++- src/bin/pg_controldata/t/001_pg_controldata.pl | 14 ++++++++++---- 2 files changed, 18 insertions(+), 5 deletions(-)
В списке pgsql-committers по дате отправления: