Skip to content

[CL-130] Record EL-rebuilt columns only after successful cache writes - #253

Open
Bronek wants to merge 2 commits into
mainfrom
bronek/fix_el_blobs_missing_column
Open

Bronek wants to merge 2 commits into
mainfrom
bronek/fix_el_blobs_missing_column

Conversation

@Bronek

@Bronek Bronek commented Sep 14, 2026

Copy link
Copy Markdown
Collaborator

Set each built bit after writing the sidecar and emitting Persist. Failed reservations or writes leave the column missing from the tracker, preventing availability and custody completion from counting unwritten data.

Add a tile test covering a cache too small for one sidecar and a cache large enough for the custody columns. Check persistence events, availability, custody completion, and missing columns in both cases.

Give the block fixture one complete commitment so it triggers an EL fetch.

Fixes: CL-130

@Bronek Bronek changed the title Record EL-rebuilt columns only after successful cache writes [CL-130] Record EL-rebuilt columns only after successful cache writes Sep 15, 2026
@Bronek
Bronek force-pushed the bronek/fix_el_blobs_missing_column branch from 64415f9 to df512dc Compare September 15, 2026 12:01
Set each built bit after writing the sidecar and emitting Persist.
Failed reservations or writes leave the column missing from the tracker,
preventing availability and custody completion from counting unwritten data.

Add a tile test covering a cache too small for one sidecar and a cache
large enough for the custody columns. Check persistence events,
availability, custody completion, and missing columns in both cases.

Give the block fixture one complete commitment so it triggers an EL fetch.

Assisted-by: Claude:claude-fable-5-1
Assisted-by: Codex:gpt-6-astra
@Bronek
Bronek force-pushed the bronek/fix_el_blobs_missing_column branch from df512dc to 5412bd3 Compare September 15, 2026 12:01
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