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,

Erik Rijkers


Reply via email to