1
73%
BABEL_5_X_DEV: 77%

Ran 23 Oct 2024 09:48AM UTC

Files 114

Run time 4s

Badge

Embed ▾

Committed 23 Oct 2024 08:09AM UTC coverage: 73.719%. Remained the same

Job # 11476747325.1

Build Type

push

github

Committed by

web-flow

Commit Message

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

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

43049 of 58396 relevant lines covered (73.72%)

142590.8 hits per line

babelfish-for-postgresql / babelfish_extensions / 11476747325 / 1
73%
BABEL_5_X_DEV: 77%

README BADGES
x

Markdown

Textile

RDoc

HTML

Rst

Source Files on job 11476747325.1

babelfish-for-postgresql / babelfish_extensions / 11476747325 / 1 73% BABEL_5_X_DEV: 77%

README BADGES x

Markdown

Textile

RDoc

HTML

Rst

Source Files on job 11476747325.1

babelfish-for-postgresql / babelfish_extensions / 11476747325 / 1
73%
BABEL_5_X_DEV: 77%

README BADGES
x