Conversation
vklachkov
force-pushed
the
corrupted-storages
branch
from
e5feef3 to
710c8ce
Compare
|
And how it's after PR: ERROR baedeker: runtime error: failed to rebuild storage LastRateLimitedBlock: failed to decode OwnerHyperparamUpdate variant: broken field 0: codec: not enough bytes to decode u16: expected 2 bytes, but only 0 byte is available Вообще там под ошибкой ещё стектрейс есть, и из него большая часть информации о пути ошибки доступна, в него же и надо дополнительную информацию внедрять По аналогии с context() из anyhow используй метод description() на Resultах |
… for baedeker integration
vklachkov
force-pushed
the
corrupted-storages
branch
from
710c8ce to
3889d7f
Compare
crates/chainql-core/src/lib.rs
Outdated
| where | ||
| Self: parity_scale_codec::Decode, | ||
| { | ||
| fn decode<I: Input>(input: &mut I) -> Result<Self> { |
There was a problem hiding this comment.
Переименовать бы, что бы было понятно что это не из оригинального Decode метод
There was a problem hiding this comment.
Специально не переименовывал, чтобы не было никакой разницы с оригинальным Decode
There was a problem hiding this comment.
Но она есть, нужно дёргать этот метод а не оригинальный decode)
CertainLach
left a comment
CertainLach
left a comment
There was a problem hiding this comment.
Кроме двух nitpicks всё хорошо, через пару часов подготовлю релиз chainql для baedeker, если не успеешь их исправить - исправлю сам когда буду версию бампать и т.д
2 participants
Blocked by #11.
Adds an option to "repair" storage with a default value and significantly improves logging for decoding errors.