Fix the error message when failing to restore the snapshot
Zhijie Hou (Fujitsu) <houzj.fnst@fujitsu.com>
From: "Zhijie Hou (Fujitsu)" <houzj.fnst@fujitsu.com>
To: PostgreSQL Hackers <pgsql-hackers@lists.postgresql.org>
Date: 2023-08-22T12:39:09Z
Lists: pgsql-hackers
Attachments
- 0001-Fix-the-error-message-when-failing-to-restore-the-sn.patch (application/octet-stream) patch 0001
Hi,
While testing the logical snapshot restore functionality, I noticed the
data size reported in the error message seems not correct.
I think it's because we used a const value here:
SnapBuildRestoreContents(int fd, char *dest, Size size, const char *path)
...
readBytes = read(fd, dest, size);
pgstat_report_wait_end();
if (readBytes != size)
...
ereport(ERROR,
(errcode(ERRCODE_DATA_CORRUPTED),
errmsg("could not read file \"%s\": read %d of %zu",
** path, readBytes, * sizeof(SnapBuild) *)));
I think we need to pass the size here.
Attach a small patch to fix this. BTW, the error message exists in HEAD ~ PG10.
Best Regards,
Hou zj
Commits
-
Fix the error message when failing to restore the snapshot.
- 13e2665df3d0 16.0 landed
- 27449ccc4d3a 17.0 landed