From: | Simon Riggs <simon(dot)riggs(at)enterprisedb(dot)com> |
---|---|
To: | Erik Rijkers <er(at)xs4all(dot)nl> |
Cc: | PostgreSQL Hackers <pgsql-hackers(at)postgresql(dot)org> |
Subject: | Re: Code checks for App Devs, using new options for transaction behavior |
Date: | 2022-10-28 09:33:25 |
Message-ID: | CANbhV-HxRx1Q1w-x-Vjjv1yfy=HbCQ+Y-tGZdUzdLS5WpcMvCA@mail.gmail.com |
Views: | Raw Message | Whole Thread | Download mbox | Resend email |
Thread: | |
Lists: | pgsql-hackers |
On Fri, 28 Oct 2022 at 07:54, Erik Rijkers <er(at)xs4all(dot)nl> wrote:
>
> Op 27-10-2022 om 18:35 schreef Simon Riggs:
> > On Thu, 27 Oct 2022 at 12:09, Simon Riggs <simon(dot)riggs(at)enterprisedb(dot)com> wrote:
> >
> >> Comments please
> >
> > Update from patch tester results.
> >
>
> > [001_psql_parse_only.v1.patch ]
> > [002_nested_xacts.v7.patch ]
> > [003_rollback_on_commit.v1.patch ]
> > [004_add_params_to_sample.v1.patch]
>
>
> patch 002 has (2x) :
> 'transction' should be
> 'transaction'
>
> also in patch 002:
> 'at any level will be abort' should be
> 'at any level will abort'
>
> I also dislike the 'we' in
>
> 'Once we reach the top-level transaction,'
>
> That seems a bit too much like the 'we developers working together to
> make a database server system' which is of course used often and
> usefully on this mailinglist and in code itself. But I think
> user-facing docs should be careful with that team-building 'we'. I
> remember well how it confused me, many years ago. Better, IMHO:
>
> 'Once the top-level transaction is reached,'
Thanks for the feedback, I will make all of those corrections in the
next version.
I'm guessing you like the features??
--
Simon Riggs http://www.EnterpriseDB.com/
From | Date | Subject | |
---|---|---|---|
Next Message | shiy.fnst@fujitsu.com | 2022-10-28 09:34:16 | RE: Perform streaming logical transactions by background workers and parallel apply |
Previous Message | Michael Paquier | 2022-10-28 08:00:23 | Re: Use pg_pwritev_with_retry() instead of write() in dir_open_for_write() to avoid partial writes? |