Horton

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

Amit Kapila <amit.kapila16@gmail.com>

View thread
From: Amit Kapila <amit.kapila16@gmail.com>
To: "Zhijie Hou (Fujitsu)" <houzj.fnst@fujitsu.com>
Cc: Kyotaro Horiguchi <horikyota.ntt@gmail.com>, "pgsql-hackers@lists.postgresql.org" <pgsql-hackers@lists.postgresql.org>
Date: 2025-09-18T03:13:48Z
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.

On Wed, Sep 17, 2025 at 8:53 AM Zhijie Hou (Fujitsu)
<houzj.fnst@fujitsu.com> wrote:
>
> >
> > In the above sentence, has advanced sounds like we have already advanced
> > but that is not the case. Also, use of into looks odd to me.
> > How about changing it to: "Retention is re-enabled because the apply process
> > can advance its xmin within the configured max_retention_duration of %u
> > ms."?
> >
> > Similarly for the first message, how about: "Retention is stopped because the
> > apply process could not advance its xmin within the configured
> > max_retention_duration of %u ms."?
>
> I think the suggested message aligns better with the implementation.
>
> I updated the message based on Horiguchi-San's revision. Additionally,
> 035_conflicts.pl has been modified, as it was waiting for the resume DETAIL
> message and this message has now been updated.
>

LGTM. Horiguchi-San, do let me know if you have suggestions here. I am
planning to push this tomorrow.

-- 
With Regards,
Amit Kapila.