Re: renaming "transaction log"

From: Alvaro Herrera <alvherre(at)2ndquadrant(dot)com>
To: David Fetter <david(at)fetter(dot)org>
Cc: Peter Eisentraut <peter(dot)eisentraut(at)2ndquadrant(dot)com>, pgsql-hackers <pgsql-hackers(at)postgresql(dot)org>
Subject: Re: renaming "transaction log"
Date: 2017-05-03 17:48:06
Message-ID: 20170503174806.2xgwburmupl2p4vu@alvherre.pgsql
Views: Raw Message | Whole Thread | Download mbox | Resend email
Thread:
Lists: pgsql-hackers

David Fetter wrote:
> On Wed, May 03, 2017 at 10:33:32AM -0700, David Fetter wrote:
> > On Wed, May 03, 2017 at 10:57:06AM -0300, Alvaro Herrera wrote:
> > > Peter Eisentraut wrote:
> > > > Most documentation and error messages still uses the term "transaction
> > > > log" to refer to the write-ahead log. Here is a patch to rename that,
> > > > which I think should be done, to match the xlog -> wal renaming in APIs.
> > >
> > > +1 for the idea. I suggest grepping for "transaction$" too -- there are
> > > a few cases in SGML that have the phrase "transaction log" split across
> > > lines.
> >
> > In general, you can use ag a.k.a. "The Silver Searcher" for phrase
> > searches as follows:

> ag $(echo "the phrase to be searched" | sed -e 's/ /\\s+/g')

Nice, thanks. So after applying Peter's patch, this grep returns the
following (after removing the *.po files from the result):

src/backend/access/transam/transam.c:4: * postgres transaction log interface routines

Probably change to "commit log"; this one is not related to WAL.

doc/src/sgml/release-old.sgml:631:<listitem><para>Fix race condition in transaction log management</para>
doc/src/sgml/release-old.sgml:2245:<listitem><para>Prevent possible compressed transaction log loss (Tom)</para></listitem>
doc/src/sgml/release-old.sgml:2282:<listitem><para>Fix for compressed transaction log id wraparound (Tom)</para></listitem>
doc/src/sgml/release-8.2.sgml:5163: Allow a forced switch to a new transaction log file (Simon, Tom)
doc/src/sgml/release-8.2.sgml:5168: in sync with the master. Transaction log file switching now also happens
doc/src/sgml/release-8.2.sgml:5172: transaction log files needed for recovery can be archived immediately.
doc/src/sgml/release-8.2.sgml:5182: Add functions for interrogating the current transaction log insertion
doc/src/sgml/release-8.2.sgml:5222: to force transaction log file switches at a given interval (Simon)
doc/src/sgml/release-8.0.sgml:2582:<listitem><para>Fix race condition in transaction log management</para>
doc/src/sgml/release-9.6.sgml:3180: to expose the current transaction log flush location (Tomas Vondra)
doc/src/sgml/release-7.4.sgml:1950:<listitem><para>Fix race condition in transaction log management</para>

Leave old release notes untouched.

doc/src/sgml/release-10.sgml:2362: 2016-11-09 [41124a91e] pgbench: Allow the transaction log file prefix to be cha
doc/src/sgml/release-10.sgml:2520: 2016-10-23 [56c7d8d45] Allow pg_basebackup to stream transaction log in tar mod

Comments, ignore. (First one is unrelated anyway.)

doc/src/sgml/high-availability.sgml:141: <term>Transaction Log Shipping</term>
doc/src/sgml/high-availability.sgml:292: <entry>Transaction Log Shipping</entry>

I think it'd make sense to flip these two to Write-Ahead Log Shipping
too. (Why aren't there index entries for the term? Maybe add two
entries, one for "transaction log" and another one for WAL).

doc/src/sgml/wal.sgml:241: <primary>transaction log</primary>

Leave this one alone, add another indexentry for the WAL terminology.

doc/src/sgml/ref/pg_receivewal.sgml:35: from a running <productname>PostgreSQL</productname> cluster. The transaction
doc/src/sgml/ref/pg_receivewal.sgml:36: log is streamed using the streaming replication protocol, and is written

Peter missed this one.

doc/src/sgml/ref/pg_receivewal.sgml:43: <application>pg_receivewal</application> streams the transaction
doc/src/sgml/ref/pg_receivewal.sgml:44: log in real time as it's being generated on the server, and does not wait

Ditto.

doc/src/sgml/ref/pg_basebackup.sgml:181: the cluster has no additional tablespaces and transaction
doc/src/sgml/ref/pg_basebackup.sgml:182: log streaming is not used.

Ditto.

doc/src/sgml/ref/pgbench.sgml:1126: <title>Per-Transaction Logging</title>

Unrelated, don't change.

--
Álvaro Herrera https://www.2ndQuadrant.com/
PostgreSQL Development, 24x7 Support, Remote DBA, Training & Services

In response to

Responses

Browse pgsql-hackers by date

  From Date Subject
Next Message Kenneth Marshall 2017-05-03 18:22:06 Re: CTE inlining
Previous Message Kevin Grittner 2017-05-03 17:47:55 Re: How huge does mvtest_huge need to be?