diff options
author | Fujii Masao <fujii@postgresql.org> | 2016-12-05 20:29:41 +0900 |
---|---|---|
committer | Fujii Masao <fujii@postgresql.org> | 2016-12-05 20:29:41 +0900 |
commit | 5dc851afde8d9ef9947f21799f7a1b08bf0bf812 (patch) | |
tree | 9b6c137056f505382dfbf753279cb406e8576b46 /src/backend/utils/misc/backend_random.c | |
parent | 3850723208888a6fe90b75ebf692af79119f4158 (diff) | |
download | postgresql-5dc851afde8d9ef9947f21799f7a1b08bf0bf812.tar.gz postgresql-5dc851afde8d9ef9947f21799f7a1b08bf0bf812.zip |
Fix incorrect output from gin_desc().
Previously gin_desc() displayed incorrect output "unknown action 0"
for XLOG_GIN_INSERT and XLOG_GIN_VACUUM_DATA_LEAF_PAGE records with
valid actions. The cause of this problem was that gin_desc() wrongly
used XLogRecGetData() to extract data from those records.
Since they were registered by XLogRegisterBufData(), gin_desc() should
have used XLogRecGetBlockData(), instead, like gin_redo().
Also there were other differences about how to treat XLOG_GIN_INSERT
record between gin_desc() and gin_redo().
This commit fixes gin_desc() routine so that it treats those records
in the same way as gin_redo().
Batch-patch to 9.5 where WAL record format was revamped and
XLogRegisterBufData() was added.
Reported-By: Andres Freund
Reviewed-By: Tom Lane
Discussion: <20160509194645.7lewnpw647zegx2m@alap3.anarazel.de>
Diffstat (limited to 'src/backend/utils/misc/backend_random.c')
0 files changed, 0 insertions, 0 deletions