1
74%
BABEL_5_X_DEV: 77%

Ran 12 Sep 2025 10:35AM UTC

Files 115

Run time 6s

Badge

Embed ▾

Committed 12 Sep 2025 10:04AM UTC coverage: 74.188%. Remained the same

Job # 17671290798.1

Build Type

push

github

Committed by

web-flow

Commit Message

Fix rollback to savepoint in TDS TxnMgmtRequest when XID is not assigned to top transaction (#4086)

In TDS TxnMgmtRequest, after we perform rollback , we check if its was rollback
to savepoint or a complete rollback. In case of compelte rollback we will send a
token back to client while incase of rollback to savepoint we do not. The check
for this relied on top transaction XID being valid if the top transaction is still
active meaning we did a rollback to savepoint but postgres only assings the XID
to transaction when it first needs it i.e. lazy assignment. Now for cases where
top transaction is XID less and we do a rollback to savepoint, the check fails to
detect that top transaction is still active and sends a token back to client anyway
leading to client error "This SqlTransaction has completed; it is no longer usable."

As a fix modify the condition to use IsTransactionBlock() instead of XID

Cherry pick of #4083

Task: BABEL-6090

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

Run Details

43927 of 59210 relevant lines covered (74.19%)

273479.39 hits per line

babelfish-for-postgresql / babelfish_extensions / 17671290798 / 1
74%
BABEL_5_X_DEV: 77%

README BADGES
x

Markdown

Textile

RDoc

HTML

Rst

Source Files on job 17671290798.1

babelfish-for-postgresql / babelfish_extensions / 17671290798 / 1 74% BABEL_5_X_DEV: 77%

README BADGES x

Markdown

Textile

RDoc

HTML

Rst

Source Files on job 17671290798.1

babelfish-for-postgresql / babelfish_extensions / 17671290798 / 1
74%
BABEL_5_X_DEV: 77%

README BADGES
x