Skip to content

Backport to 2.28.x: #10242: Fix incremental refresh skipping the last bucket - #10294

Open
timescale-automation wants to merge 1 commit into
2.28.xfrom
backport/2.28.x/10242
Open

Backport to 2.28.x: #10242: Fix incremental refresh skipping the last bucket#10294
timescale-automation wants to merge 1 commit into
2.28.xfrom
backport/2.28.x/10242

Conversation

@timescale-automation

Copy link
Copy Markdown
Member

This is an automated backport of #10242: Fix incremental refresh skipping the last bucket.
The original issue is #10221.
This PR will be merged automatically after all the relevant CI checks pass. If this fix should not be backported, or will be backported manually, just close this PR. You can use the backport branch to add your changes, it won't be modified automatically anymore.

For more details, please see the documentation

Original description

Fix incremental refresh skipping the last bucket

An invalidation ending exactly on a bucket start was retired one bucket too early, so that bucket never got a batch and its entry was left stranded in the materialization invalidation log, silently missing from query results until a later refresh covered the region.

Treat greatest_modified_value of an invalidation as an inclusive bound to include the missing bucket.

Fixes #10221

An invalidation ending exactly on a bucket start was retired one bucket
too early, so that bucket never got a batch and its entry was left
stranded in the materialization invalidation log, silently missing from
query results until a later refresh covered the region.

Treat greatest_modified_value of an invalidation as an inclusive bound
to include the missing bucket.

Fixes #10221

(cherry picked from commit 577f1cf)
@codecov

codecov Bot commented Jul 20, 2026

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.

📢 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

is-auto-backport PR created by backport automation

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants