On Fri, 28 Oct 2022 at 07:54, Erik Rijkers <er@xs4all.nl> wrote:
>
> Op 27-10-2022 om 18:35 schreef Simon Riggs:
> > On Thu, 27 Oct 2022 at 12:09, Simon Riggs <simon.riggs@enterprisedb.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/