Skip to content

node, store: Remove unused ChainStatus enum.#6385

Merged
dimitrovmaksim merged 1 commit intographprotocol:masterfrom
dimitrovmaksim:remove-unused-chain-status
Feb 19, 2026
Merged

node, store: Remove unused ChainStatus enum.#6385
dimitrovmaksim merged 1 commit intographprotocol:masterfrom
dimitrovmaksim:remove-unused-chain-status

Conversation

@dimitrovmaksim
Copy link
Member

Closes #6183

  • Removes unused ChainStatus enum
  • Changes the shard configuration mismatch log from error to warn

@dimitrovmaksim dimitrovmaksim self-assigned this Feb 18, 2026
if chain.shard != shard {
warn!(
&block_store.logger,
"the chain {} is stored in shard {} but is configured for shard {}",
Copy link
Collaborator

Choose a reason for hiding this comment

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

The log message would be even clearer if it said

"the chain {} is stored in shard {} but is configured for shard {}; ignoring config and using shard {chain.shard}"

Copy link
Member Author

Choose a reason for hiding this comment

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

Updated and force pushed 🫡

@dimitrovmaksim dimitrovmaksim force-pushed the remove-unused-chain-status branch from e78dd20 to bbe82db Compare February 19, 2026 07:57
@dimitrovmaksim dimitrovmaksim force-pushed the remove-unused-chain-status branch from 67ada46 to 7345657 Compare February 19, 2026 11:20
@dimitrovmaksim dimitrovmaksim merged commit a5c8615 into graphprotocol:master Feb 19, 2026
6 checks passed
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.

[Bug] the chain NNN is stored in shard XXX but is configured for shard YYY

2 participants

Comments