babelfish-for-postgresql / babelfish_extensions / 11464483590

76%

BABEL_5_X_DEV: 77%

Build Type

push

github

Committed by web-flow

Commit Message

cleanup pltsql state properly in exec_stmt_exec_batch in case of errors (#3036)

Some system states like pltsql guc nest level, pltsql scope identity should always be cleaned up at the end of execution even in case of errors since these were defined to be outside of postgres's transaction cleanup control.
So a pltsql_new_guc_nest_level() should always have a pltsql_revert_guc in all cases and vice versa.

This was not happening inside exec_stmt_exec_batch where we could error out even before starting a new pltsql guc nest level and then in catch block we would try to revert a non existent nest level.

As a fix we move out the call to pltsql_new_guc_nest_level() before try block ensuring we have a nest level to revert in catch.

Task: BABEL-5344
Signed-off-by: Tanzeel Khan <tzlkhan@amazon.com>

Run Details

8 of 8 new or added lines in 1 file covered. (100.0%)

1 existing line in 1 file now uncovered.

45232 of 60686 relevant lines covered (74.53%)

162839.0 hits per line