Skip to content

node, store: Remove unused ChainStatus enum.#6385

Open
dimitrovmaksim wants to merge 1 commit intographprotocol:masterfrom
dimitrovmaksim:remove-unused-chain-status
Open

node, store: Remove unused ChainStatus enum.#6385
dimitrovmaksim wants to merge 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}"

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