Horton

Re: Reword messages using "as" instead of "because"

Tom Lane <tgl@sss.pgh.pa.us>

View thread
From: Tom Lane <tgl@sss.pgh.pa.us>
To: Amit Kapila <amit.kapila16@gmail.com>
Cc: "Zhijie Hou (Fujitsu)" <houzj.fnst@fujitsu.com>, Kyotaro Horiguchi <horikyota.ntt@gmail.com>, "pgsql-hackers@lists.postgresql.org" <pgsql-hackers@lists.postgresql.org>
Date: 2025-09-19T03:43:34Z
Lists: pgsql-hackers

Commits

Same data as JSON: GET /api/v1/messages/:b64id/commits the thread's linked commits as JSON, with link sources. API reference →

  1. Improve few errdetail messages introduced in commit 0d48d393d46.

  2. Resume conflict-relevant data retention automatically.

Amit Kapila <amit.kapila16@gmail.com> writes:
> We have a similar message for stop retention. I feel it would be good
> to mention that as a reason, so users can increase it. I could think
> of two alternatives for stop message based on above suggestion:
> "Retention is stopped because the apply process has not caught up with
> the publisher within the configured max_retention_duration."
> "Retention is stopped because the apply process could not catch up
> with the publisher within the configured max_retention_duration."

> Do you have any preference?

I think "has not" is clearer, or maybe you should say "did not catch
up with..."  Either way, that sounds like a pure statement of fact
whereas "could not" has some overtones of assigning blame.

			regards, tom lane