From 6094c80f28c64904ede361672521be6b86993adb Mon Sep 17 00:00:00 2001 From: Bharath Rupireddy Date: Wed, 20 Jul 2022 04:25:10 +0000 Subject: [PATCH v7] Use "WAL segment" instead of "log segment" We are using "log segment" in various user-facing messages, the term "log" can mean server logs as well. The "WAL segment" suits well here and it is consistently used across the other user-facing messages. Author: Bharath Rupireddy --- src/backend/access/transam/xlogreader.c | 10 +++++----- src/backend/access/transam/xlogrecovery.c | 6 +++--- src/backend/access/transam/xlogutils.c | 4 ++-- src/backend/replication/walreceiver.c | 6 +++--- src/bin/pg_resetwal/pg_resetwal.c | 2 +- src/bin/pg_upgrade/controldata.c | 2 +- src/bin/pg_waldump/pg_waldump.c | 4 ++-- 7 files changed, 17 insertions(+), 17 deletions(-) diff --git a/src/backend/access/transam/xlogreader.c b/src/backend/access/transam/xlogreader.c index f3dc4b7797..58f1a32b00 100644 --- a/src/backend/access/transam/xlogreader.c +++ b/src/backend/access/transam/xlogreader.c @@ -1209,7 +1209,7 @@ XLogReaderValidatePageHeader(XLogReaderState *state, XLogRecPtr recptr, XLogFileName(fname, state->seg.ws_tli, segno, state->segcxt.ws_segsize); report_invalid_record(state, - "invalid magic number %04X in log segment %s, offset %u", + "invalid magic number %04X in WAL segment %s, offset %u", hdr->xlp_magic, fname, offset); @@ -1223,7 +1223,7 @@ XLogReaderValidatePageHeader(XLogReaderState *state, XLogRecPtr recptr, XLogFileName(fname, state->seg.ws_tli, segno, state->segcxt.ws_segsize); report_invalid_record(state, - "invalid info bits %04X in log segment %s, offset %u", + "invalid info bits %04X in WAL segment %s, offset %u", hdr->xlp_info, fname, offset); @@ -1264,7 +1264,7 @@ XLogReaderValidatePageHeader(XLogReaderState *state, XLogRecPtr recptr, /* hmm, first page of file doesn't have a long header? */ report_invalid_record(state, - "invalid info bits %04X in log segment %s, offset %u", + "invalid info bits %04X in WAL segment %s, offset %u", hdr->xlp_info, fname, offset); @@ -1283,7 +1283,7 @@ XLogReaderValidatePageHeader(XLogReaderState *state, XLogRecPtr recptr, XLogFileName(fname, state->seg.ws_tli, segno, state->segcxt.ws_segsize); report_invalid_record(state, - "unexpected pageaddr %X/%X in log segment %s, offset %u", + "unexpected pageaddr %X/%X in WAL segment %s, offset %u", LSN_FORMAT_ARGS(hdr->xlp_pageaddr), fname, offset); @@ -1308,7 +1308,7 @@ XLogReaderValidatePageHeader(XLogReaderState *state, XLogRecPtr recptr, XLogFileName(fname, state->seg.ws_tli, segno, state->segcxt.ws_segsize); report_invalid_record(state, - "out-of-sequence timeline ID %u (after %u) in log segment %s, offset %u", + "out-of-sequence timeline ID %u (after %u) in WAL segment %s, offset %u", hdr->xlp_tli, state->latestPageTLI, fname, diff --git a/src/backend/access/transam/xlogrecovery.c b/src/backend/access/transam/xlogrecovery.c index 5d6f1b5e46..306a9f40e9 100644 --- a/src/backend/access/transam/xlogrecovery.c +++ b/src/backend/access/transam/xlogrecovery.c @@ -3018,7 +3018,7 @@ ReadRecord(XLogPrefetcher *xlogprefetcher, int emode, XLogFileName(fname, xlogreader->seg.ws_tli, segno, wal_segment_size); ereport(emode_for_corrupt_record(emode, xlogreader->EndRecPtr), - (errmsg("unexpected timeline ID %u in log segment %s, offset %u", + (errmsg("unexpected timeline ID %u in WAL segment %s, offset %u", xlogreader->latestPageTLI, fname, offset))); @@ -3223,13 +3223,13 @@ retry: errno = save_errno; ereport(emode_for_corrupt_record(emode, targetPagePtr + reqLen), (errcode_for_file_access(), - errmsg("could not read from log segment %s, offset %u: %m", + errmsg("could not read from WAL segment %s, offset %u: %m", fname, readOff))); } else ereport(emode_for_corrupt_record(emode, targetPagePtr + reqLen), (errcode(ERRCODE_DATA_CORRUPTED), - errmsg("could not read from log segment %s, offset %u: read %d of %zu", + errmsg("could not read from WAL segment %s, offset %u: read %d of %zu", fname, readOff, r, (Size) XLOG_BLCKSZ))); goto next_record_is_invalid; } diff --git a/src/backend/access/transam/xlogutils.c b/src/backend/access/transam/xlogutils.c index 0cda22597f..9e3a000768 100644 --- a/src/backend/access/transam/xlogutils.c +++ b/src/backend/access/transam/xlogutils.c @@ -1049,14 +1049,14 @@ WALReadRaiseError(WALReadError *errinfo) errno = errinfo->wre_errno; ereport(ERROR, (errcode_for_file_access(), - errmsg("could not read from log segment %s, offset %d: %m", + errmsg("could not read from WAL segment %s, offset %d: %m", fname, errinfo->wre_off))); } else if (errinfo->wre_read == 0) { ereport(ERROR, (errcode(ERRCODE_DATA_CORRUPTED), - errmsg("could not read from log segment %s, offset %d: read %d of %d", + errmsg("could not read from WAL segment %s, offset %d: read %d of %d", fname, errinfo->wre_off, errinfo->wre_read, errinfo->wre_req))); } diff --git a/src/backend/replication/walreceiver.c b/src/backend/replication/walreceiver.c index 3d37c1fe62..3767466ef3 100644 --- a/src/backend/replication/walreceiver.c +++ b/src/backend/replication/walreceiver.c @@ -616,7 +616,7 @@ WalReceiverMain(void) if (close(recvFile) != 0) ereport(PANIC, (errcode_for_file_access(), - errmsg("could not close log segment %s: %m", + errmsg("could not close WAL segment %s: %m", xlogfname))); /* @@ -930,7 +930,7 @@ XLogWalRcvWrite(char *buf, Size nbytes, XLogRecPtr recptr, TimeLineID tli) errno = save_errno; ereport(PANIC, (errcode_for_file_access(), - errmsg("could not write to log segment %s " + errmsg("could not write to WAL segment %s " "at offset %u, length %lu: %m", xlogfname, startoff, (unsigned long) segbytes))); } @@ -1042,7 +1042,7 @@ XLogWalRcvClose(XLogRecPtr recptr, TimeLineID tli) if (close(recvFile) != 0) ereport(PANIC, (errcode_for_file_access(), - errmsg("could not close log segment %s: %m", + errmsg("could not close WAL segment %s: %m", xlogfname))); /* diff --git a/src/bin/pg_resetwal/pg_resetwal.c b/src/bin/pg_resetwal/pg_resetwal.c index d4772a2965..7adf79eeed 100644 --- a/src/bin/pg_resetwal/pg_resetwal.c +++ b/src/bin/pg_resetwal/pg_resetwal.c @@ -788,7 +788,7 @@ PrintNewControlValues(void) XLogFileName(fname, ControlFile.checkPointCopy.ThisTimeLineID, newXlogSegNo, WalSegSz); - printf(_("First log segment after reset: %s\n"), fname); + printf(_("First WAL segment after reset: %s\n"), fname); if (set_mxid != 0) { diff --git a/src/bin/pg_upgrade/controldata.c b/src/bin/pg_upgrade/controldata.c index 07de918358..678e8ebf6b 100644 --- a/src/bin/pg_upgrade/controldata.c +++ b/src/bin/pg_upgrade/controldata.c @@ -350,7 +350,7 @@ get_control_data(ClusterInfo *cluster, bool live_check) cluster->controldata.chkpnt_nxtmxoff = str2uint(p); got_mxoff = true; } - else if ((p = strstr(bufin, "First log segment after reset:")) != NULL) + else if ((p = strstr(bufin, "First WAL segment after reset:")) != NULL) { /* Skip the colon and any whitespace after it */ p = strchr(p, ':'); diff --git a/src/bin/pg_waldump/pg_waldump.c b/src/bin/pg_waldump/pg_waldump.c index 6528113628..4eebeadc8c 100644 --- a/src/bin/pg_waldump/pg_waldump.c +++ b/src/bin/pg_waldump/pg_waldump.c @@ -667,7 +667,7 @@ usage(void) printf(_(" -F, --fork=FORK only show records that modify blocks in fork FORK;\n" " valid names are main, fsm, vm, init\n")); printf(_(" -n, --limit=N number of records to display\n")); - printf(_(" -p, --path=PATH directory in which to find log segment files or a\n" + printf(_(" -p, --path=PATH directory in which to find WAL segment files or a\n" " directory with a ./pg_wal that contains such files\n" " (default: current directory, ./pg_wal, $PGDATA/pg_wal)\n")); printf(_(" -q, --quiet do not print any output, except for errors\n")); @@ -675,7 +675,7 @@ usage(void) " use --rmgr=list to list valid resource manager names\n")); printf(_(" -R, --relation=T/D/R only show records that modify blocks in relation T/D/R\n")); printf(_(" -s, --start=RECPTR start reading at WAL location RECPTR\n")); - printf(_(" -t, --timeline=TLI timeline from which to read log records\n" + printf(_(" -t, --timeline=TLI timeline from which to read WAL records\n" " (default: 1 or the value used in STARTSEG)\n")); printf(_(" -V, --version output version information, then exit\n")); printf(_(" -w, --fullpage only show records with a full page write\n")); -- 2.25.1