Skip to content

fix: add optimistic progress to indexing status report#887

Merged
Theodus merged 2 commits intomainfrom
theodus/block-range
Jul 16, 2024
Merged

fix: add optimistic progress to indexing status report#887
Theodus merged 2 commits intomainfrom
theodus/block-range

Conversation

@Theodus
Copy link
Member

@Theodus Theodus commented Jul 16, 2024

This avoids an edge case where users receive a block number from an indexer but then fail to make a subsequent query to the same indexer because we think it hasn't hit that block yet.

@Theodus Theodus requested a review from LNSD July 16, 2024 12:16
Copy link
Contributor

@LNSD LNSD left a comment

Choose a reason for hiding this comment

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

LGTM ✅

@Theodus Theodus merged commit 7a3bb3b into main Jul 16, 2024
@Theodus Theodus deleted the theodus/block-range branch July 16, 2024 15:53
@Theodus Theodus mentioned this pull request Jul 22, 2024
Theodus added a commit that referenced this pull request Jul 22, 2024
# Release Notes
- feat: add to indexer & client query kafka messages (#879)
- fix: add optimistic progress to indexing status report (#887)
- feat: add rate limiter for L1 deprecation (#889, #898)
- fix: handle indexer-service error format (#896)
- fix: remove block constraint rewrites (#895)
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.

2 participants