From: Alvaro Herrera Date: Mon, 11 Feb 2019 18:55:09 +0000 (-0300) Subject: Fix misleading PG_RE_THROW commentary X-Git-Tag: REL_12_BETA1~767 X-Git-Url: http://git.ipfire.org/?a=commitdiff_plain;h=c603b392c32513982439bc466312d3a6b697d53e;p=thirdparty%2Fpostgresql.git Fix misleading PG_RE_THROW commentary The old verbiage indicated that PG_RE_THROW is optional, which is not really true. This has confused many people, so it seems worth fixing. Discussion: https://postgr.es/m/20190206160958.GA22304@alvherre.pgsql --- diff --git a/src/include/utils/elog.h b/src/include/utils/elog.h index 648eedf5af7..7ac37fd270e 100644 --- a/src/include/utils/elog.h +++ b/src/include/utils/elog.h @@ -272,8 +272,10 @@ extern PGDLLIMPORT ErrorContextCallback *error_context_stack; * PG_END_TRY(); * * (The braces are not actually necessary, but are recommended so that - * pgindent will indent the construct nicely.) The error recovery code - * can optionally do PG_RE_THROW() to propagate the same error outwards. + * pgindent will indent the construct nicely.) The error recovery code + * can either do PG_RE_THROW to propagate the error outwards, or do a + * (sub)transaction abort. Failure to do so may leave the system in an + * inconsistent state for further processing. * * Note: while the system will correctly propagate any new ereport(ERROR) * occurring in the recovery section, there is a small limit on the number