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

Better metrics & logs around indexing health / ETA #3265

Open
Tracked by #3414
tkporter opened this issue Feb 15, 2024 · 0 comments
Open
Tracked by #3414

Better metrics & logs around indexing health / ETA #3265

tkporter opened this issue Feb 15, 2024 · 0 comments
Assignees

Comments

@tkporter
Copy link
Collaborator

tkporter commented Feb 15, 2024

ETA calculation is broken for some types of cursors - this ticket includes fixing all of these or deciding some alternative

  • We have the indexed_height metric which made complete sense when we just had sliding windows of blocks. However with the sequence-aware cursors, this doesn't make as much sense (or does it ?)
    • We could find an alternative metric?
    • Or for sequence-aware cursors just return the max(target snapshot block, current indexing block)
    • The indexed height probably gets messed up by rewinds?
    • Should we instead / also use timestamps as suggested here? Sequence-aware indexing refactor #3262 (comment)
  • It doesn't work for the backwards cursor, we should either implement this or justify it not existing (related: Index time to sync is incorrect for the backwards cursor #2504)
  • Log health should be really clear, both in logs and metrics. Should we have a counter for indexing rewinds or errors?
@avious00 avious00 moved this to Next Sprint in Hyperlane Tasks Feb 17, 2024
@tkporter tkporter changed the title Better metrics around indexing health Better metrics & logs around indexing health Feb 19, 2024
@tkporter tkporter changed the title Better metrics & logs around indexing health Better metrics & logs around indexing health / ETA Feb 19, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
Status: No status
Development

No branches or pull requests

2 participants