Skip to content

fix: insert splits + nextval - #1518

Merged
levkk merged 1 commit into
mainfrom
levkk-fix-insert-split-auto-id
Sep 10, 2026
Merged

fix: insert splits + nextval#1518
levkk merged 1 commit into
mainfrom
levkk-fix-insert-split-auto-id

Conversation

@levkk

@levkk levkk commented Sep 10, 2026

Copy link
Copy Markdown
Collaborator

pgdog.nextval was not being rewritten for insert splits.

@levkk
levkk merged commit a9a3c2c into main Sep 10, 2026
6 checks passed
@levkk
levkk deleted the levkk-fix-insert-split-auto-id branch September 10, 2026 15:00
@codecov

codecov Bot commented Sep 10, 2026

Copy link
Copy Markdown

Codecov Report

❌ Patch coverage is 98.76033% with 3 lines in your changes missing coverage. Please review.

Files with missing lines Patch % Lines
...frontend/router/parser/rewrite/statement/insert.rs 94.73% 2 Missing ⚠️
...c/frontend/router/parser/rewrite/statement/plan.rs 94.44% 1 Missing ⚠️

📢 Thoughts on this report? Let us know!

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