Skip to content

fix: SET TRANSACTION was captured as a session parameter - #686

Merged
levkk merged 1 commit into
mainfrom
levkk-fix-transation-state-catching-params
Dec 19, 2025
Merged

fix: SET TRANSACTION was captured as a session parameter#686
levkk merged 1 commit into
mainfrom
levkk-fix-transation-state-catching-params

Conversation

@levkk

@levkk levkk commented Dec 19, 2025

Copy link
Copy Markdown
Collaborator

Fixes

WARNING:  SET TRANSACTION can only be used in transaction blocks

and possibly the not in transaction warnings reported in #685

@codecov

codecov Bot commented Dec 19, 2025

Copy link
Copy Markdown

Codecov Report

❌ Patch coverage is 95.23810% with 1 line in your changes missing coverage. Please review.

Files with missing lines Patch % Lines
.../src/frontend/router/parser/query/test/test_set.rs 91.66% 1 Missing ⚠️

📢 Thoughts on this report? Let us know!

@levkk
levkk merged commit 77698af into main Dec 19, 2025
8 checks passed
@levkk
levkk deleted the levkk-fix-transation-state-catching-params branch December 19, 2025 18:21
dev-lew pushed a commit to dev-lew/pgdog that referenced this pull request Feb 18, 2026
Fixes

```
WARNING:  SET TRANSACTION can only be used in transaction blocks
```

and possibly the not in transaction warnings reported in pgdogdev#685
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant