Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

ChainIndexer: GetMsgInfo should return ErrNotFound when not found #12674

Open
rvagg opened this issue Nov 6, 2024 · 0 comments · May be fixed by #12680
Open

ChainIndexer: GetMsgInfo should return ErrNotFound when not found #12674

rvagg opened this issue Nov 6, 2024 · 0 comments · May be fixed by #12680
Assignees
Labels
chainindex Issues related to the new `ChainIndexer` in Lotus good first issue Good for newcomers

Comments

@rvagg
Copy link
Member

rvagg commented Nov 6, 2024

2024-11-06T21:13:14.735+1100    WARN    statemgr        stmgr/searchwait.go:175 error searching message index: error looking up message in index: sql: no rows in result set

Lots of this kind of thing in the logs, but it shouldn't be logged:

case errors.Is(err, index.ErrNotFound):
// ok for the index to have incomplete data
default:
log.Warnf("error searching message index: %s", err)
}

GetMsgInfo should return an ErrNotFound when there are no rows, here:

if err := si.queryMsgInfo(ctx, messageCid, &tipsetKeyCidBytes, &height); err != nil {
return nil, err
}

Needs tests as well.

@rvagg rvagg added good first issue Good for newcomers chainindex Issues related to the new `ChainIndexer` in Lotus labels Nov 6, 2024
@github-project-automation github-project-automation bot moved this to 📌 Triage in FilOz Nov 6, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
chainindex Issues related to the new `ChainIndexer` in Lotus good first issue Good for newcomers
Projects
Status: 📌 Triage
Development

Successfully merging a pull request may close this issue.

2 participants