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