Skip to content

Backport to branch(3.16) : Fix index-based Get/Scan to filter out results whose index key no longer matches after lazy recovery rollback#3504

Merged
brfrn169 merged 1 commit into
3.16from
3.16-pull-3488
Apr 14, 2026
Merged

Backport to branch(3.16) : Fix index-based Get/Scan to filter out results whose index key no longer matches after lazy recovery rollback#3504
brfrn169 merged 1 commit into
3.16from
3.16-pull-3488

Conversation

@feeblefakie

Copy link
Copy Markdown
Member

This is an automated backport of the following:

Please merge this PR after all checks have passed.

@gemini-code-assist gemini-code-assist Bot left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Code Review

This pull request introduces a fix for secondary index lookups within the consensus commit protocol. It addresses a specific edge case where a record retrieved via a secondary index is rolled back during lazy recovery, resulting in an index column value that no longer matches the queried key. The changes ensure these records are correctly filtered out of the final result set and prevent incorrect 'absent' results from being cached in the transaction snapshot. Comprehensive unit and integration tests have been added to verify the filtering logic across read and scan operations. I have no feedback to provide as there were no review comments to assess.

@brfrn169 brfrn169 merged commit 81c62b4 into 3.16 Apr 14, 2026
14 checks passed
@brfrn169 brfrn169 deleted the 3.16-pull-3488 branch April 14, 2026 06:24
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.

2 participants