diff options
| author | Peter Eisentraut <peter@eisentraut.org> | 2019-11-01 11:09:52 +0100 |
|---|---|---|
| committer | Peter Eisentraut <peter@eisentraut.org> | 2019-11-01 11:18:03 +0100 |
| commit | 604bd3671121b51f977de146ed95484c2297fb3e (patch) | |
| tree | 829045453c054befb9d795c33765501e2f4522f3 /src/backend/tcop/utility.c | |
| parent | 73025140885c889410b9bfc4a30a3866396fc5db (diff) | |
| download | postgresql-604bd3671121b51f977de146ed95484c2297fb3e.tar.gz postgresql-604bd3671121b51f977de146ed95484c2297fb3e.zip | |
PG_FINALLY
This gives an alternative way of catching exceptions, for the common
case where the cleanup code is the same in the error and non-error
cases. So instead of
PG_TRY();
{
... code that might throw ereport(ERROR) ...
}
PG_CATCH();
{
cleanup();
PG_RE_THROW();
}
PG_END_TRY();
cleanup();
one can write
PG_TRY();
{
... code that might throw ereport(ERROR) ...
}
PG_FINALLY();
{
cleanup();
}
PG_END_TRY();
Discussion: https://www.postgresql.org/message-id/flat/95a822c3-728b-af0e-d7e5-71890507ae0c%402ndquadrant.com
Diffstat (limited to 'src/backend/tcop/utility.c')
| -rw-r--r-- | src/backend/tcop/utility.c | 10 |
1 files changed, 2 insertions, 8 deletions
diff --git a/src/backend/tcop/utility.c b/src/backend/tcop/utility.c index c6faa6619d2..f2269ad35ca 100644 --- a/src/backend/tcop/utility.c +++ b/src/backend/tcop/utility.c @@ -1514,13 +1514,11 @@ ProcessUtilitySlow(ParseState *pstate, address = ExecRefreshMatView((RefreshMatViewStmt *) parsetree, queryString, params, completionTag); } - PG_CATCH(); + PG_FINALLY(); { EventTriggerUndoInhibitCommandCollection(); - PG_RE_THROW(); } PG_END_TRY(); - EventTriggerUndoInhibitCommandCollection(); break; case T_CreateTrigStmt: @@ -1716,16 +1714,12 @@ ProcessUtilitySlow(ParseState *pstate, EventTriggerDDLCommandEnd(parsetree); } } - PG_CATCH(); + PG_FINALLY(); { if (needCleanup) EventTriggerEndCompleteQuery(); - PG_RE_THROW(); } PG_END_TRY(); - - if (needCleanup) - EventTriggerEndCompleteQuery(); } /* |