diff options
| author | Peter Eisentraut <peter@eisentraut.org> | 2023-01-16 09:20:44 +0100 |
|---|---|---|
| committer | Peter Eisentraut <peter@eisentraut.org> | 2023-01-16 09:54:15 +0100 |
| commit | cf74b6eadb9d8fcfd51a99e9cc6b04d019e36b12 (patch) | |
| tree | c78f3dccfc1d10cc43d93d56e8a4db8676dd5903 | |
| parent | 787db4be947c2be419c4e97ba9d611708f2523c5 (diff) | |
| download | postgresql-cf74b6eadb9d8fcfd51a99e9cc6b04d019e36b12.tar.gz postgresql-cf74b6eadb9d8fcfd51a99e9cc6b04d019e36b12.zip | |
Fix some BufFileRead() error reporting
Remove "%m" from error messages where errno would be bogus. Add short
read byte counts where appropriate.
This is equivalent to what was done in
7897e3bb902c557412645b82120f4d95f7474906, but some code was apparently
developed concurrently to that and not updated accordingly.
Reviewed-by: Amit Kapila <amit.kapila16@gmail.com>
Discussion: https://www.postgresql.org/message-id/flat/f3501945-c591-8cc3-5ef0-b72a2e0eaa9c@enterprisedb.com
| -rw-r--r-- | src/backend/replication/backup_manifest.c | 3 |
1 files changed, 2 insertions, 1 deletions
diff --git a/src/backend/replication/backup_manifest.c b/src/backend/replication/backup_manifest.c index ff240c07675..5769ba3f712 100644 --- a/src/backend/replication/backup_manifest.c +++ b/src/backend/replication/backup_manifest.c @@ -358,7 +358,8 @@ SendBackupManifest(backup_manifest_info *manifest) if (rc != bytes_to_read) ereport(ERROR, (errcode_for_file_access(), - errmsg("could not read from temporary file: %m"))); + errmsg("could not read from temporary file: read only %zu of %zu bytes", + rc, bytes_to_read))); pq_putmessage('d', manifestbuf, bytes_to_read); manifest_bytes_done += bytes_to_read; } |