diff options
| author | Michael Paquier <michael@paquier.xyz> | 2023-09-14 10:30:34 +0900 |
|---|---|---|
| committer | Michael Paquier <michael@paquier.xyz> | 2023-09-14 10:30:34 +0900 |
| commit | 33bc7d4246956ada7830984d74114d3ef0f40201 (patch) | |
| tree | 415565966e4b689f5ca507f4d286a04b73d7cae3 | |
| parent | 012b80ad20c2f8459d4cbe083e3f0323050d275b (diff) | |
| download | postgresql-33bc7d4246956ada7830984d74114d3ef0f40201.tar.gz postgresql-33bc7d4246956ada7830984d74114d3ef0f40201.zip | |
Improve error message on snapshot import in snapmgr.c
When a snapshot file fails to be read in ImportSnapshot(), it would
issue an ERROR as "invalid snapshot identifier" when opening a stream
for it in read-only mode. This error message is reworded to be the same
as all the other messages used in this case on failure, which is useful
when debugging this area.
Thinko introduced by bb446b689b66 where snapshot imports have been
added. A backpatch down to 11 is done as this can improve any work
related to snapshot imports in older branches.
Author: Bharath Rupireddy
Reviewed-by: Daniel Gustafsson
Discussion: https://postgr.es/m/CALj2ACWmr=3KdxDkm8h7Zn1XxBoF6hdzq8WQyMn2y1OL5RYFrg@mail.gmail.com
Backpatch-through: 11
| -rw-r--r-- | src/backend/utils/time/snapmgr.c | 5 |
1 files changed, 3 insertions, 2 deletions
diff --git a/src/backend/utils/time/snapmgr.c b/src/backend/utils/time/snapmgr.c index 44a72d8bdac..c14741289dc 100644 --- a/src/backend/utils/time/snapmgr.c +++ b/src/backend/utils/time/snapmgr.c @@ -1540,8 +1540,9 @@ ImportSnapshot(const char *idstr) f = AllocateFile(path, PG_BINARY_R); if (!f) ereport(ERROR, - (errcode(ERRCODE_INVALID_PARAMETER_VALUE), - errmsg("invalid snapshot identifier: \"%s\"", idstr))); + (errcode_for_file_access(), + errmsg("could not open file \"%s\" for reading: %m", + path))); /* get the size of the file so that we know how much memory we need */ if (fstat(fileno(f), &stat_buf)) |