On Fri, Dec 16, 2022 at 12:49 AM Tom Lane <t...@sss.pgh.pa.us> wrote: > > Masahiko Sawada <sawada.m...@gmail.com> writes: > > I don't think we need additional PG_TRY() for that since exec_stmts() > > is already called in PG_TRY() if there is an exception block. I meant > > to call stmt_end() in PG_CATCH() in exec_stmt_block() (i.e. only when > > an error is caught by the exception block). Currently, if an error is > > caught, we call stmt_begin() and stmt_end() for statements executed > > inside the exception block but call only stmt_begin() for the > > statement that raised an error. > > I fail to see anything wrong with that. We never completed execution > of the statement that raised an error, but calling stmt_end for it > would imply that we did.
Thank you for the comment. Agreed. Regards, -- Masahiko Sawada Amazon Web Services: https://aws.amazon.com
