Skip to content

node: Validate copy offset against earliest_block_number#6384

Merged
dimitrovmaksim merged 1 commit into
graphprotocol:masterfrom
dimitrovmaksim:fix/copy-earliest-block-validation
Feb 19, 2026
Merged

node: Validate copy offset against earliest_block_number#6384
dimitrovmaksim merged 1 commit into
graphprotocol:masterfrom
dimitrovmaksim:fix/copy-earliest-block-validation

Conversation

@dimitrovmaksim

Copy link
Copy Markdown
Member

Resolves #6366

@dimitrovmaksim dimitrovmaksim force-pushed the fix/copy-earliest-block-validation branch from 267a625 to 724799d Compare February 18, 2026 19:17
@dimitrovmaksim dimitrovmaksim self-assigned this Feb 18, 2026

@lutter lutter left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

Nice!

@dimitrovmaksim dimitrovmaksim merged commit 1a47b7c into graphprotocol:master Feb 19, 2026
6 checks passed
@dimitrovmaksim dimitrovmaksim deleted the fix/copy-earliest-block-validation branch April 3, 2026 09:29
incrypto32 added a commit that referenced this pull request Jun 9, 2026
Mirrors the guard added for `graphman copy` in #6384. Without this,
grafting at a block below `base.earliest_block_number` silently produces
a destination missing all pre-graft mutable entity versions, and (when
the base has dynamic data sources) handlers fail with
`unexpected null in handler` because `data_sources$` registrations
survive pruning while the entities they reference do not.
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.

graphman copy copies pruned earliest_block_number from source, resulting in invalid destination

2 participants