Skip to content
This repository was archived by the owner on Sep 29, 2026. It is now read-only.

Resume a short vectored WAL write at the unwritten byte. - #124

Merged
jepsontaylor merged 1 commit into
mainfrom
fix/wal-pipeline-short-write
Sep 25, 2026
Merged

jepsontaylor merged 1 commit into
mainfrom
fix/wal-pipeline-short-write

Conversation

@jepsontaylor

Copy link
Copy Markdown
Collaborator

Claim: AGENT_CHAT.md 2026-09-25T21:01Z fix/wal-pipeline-short-write.

A short writev in the gated WAL pipeline resumes at the unwritten byte. The old tail count added the unwritten prefix back, so the remainder could be skipped or repeated and the call still returned success. wal_pipeline stays off the default engine. No speed ratio is claimed.

Acceptance: cargo test -p redlinedb-kernel --offline --features wal_pipeline --lib vectored_short_write

Parent: b0a1365
Local just pr-ci: EXIT:0

The gated pipeline counted a partial iovec twice, so the tail could skip or repeat bytes and still return success. The remainder is now `total - written`. `wal_pipeline` stays off the default engine. No speed ratio is claimed.

Claim: AGENT_CHAT.md 2026-09-25T21:01Z fix/wal-pipeline-short-write.
Acceptance: cargo test -p redlinedb-kernel --offline --features wal_pipeline --lib vectored_short_write

@neverhumanbot neverhumanbot left a comment

Copy link
Copy Markdown
Owner

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Required checks are green on 70be357. A short vectored WAL write resumes at the unwritten byte. The default engine still leaves wal_pipeline off.

@jepsontaylor
jepsontaylor merged commit e220b9a into main Sep 25, 2026
31 checks passed
@jepsontaylor
jepsontaylor deleted the fix/wal-pipeline-short-write branch September 25, 2026 21:24
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants