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

(state-indexer): Bug with indexing epoch validators info #160

Merged
merged 4 commits into from
Jan 19, 2024

Conversation

kobayurii
Copy link
Member

@kobayurii kobayurii commented Jan 17, 2024

The current implementation faces a challenge where information about validators during an epoch in Near operations may undergo changes. The StakeAction, acting as a trigger for these changes, is not properly considered by the indexer. As a result, the indexer neglects to update validator information when StakeAction occurs, leading to incomplete and inaccurate data about validators for that epoch.

In this PR:

  1. Configured the system to save comprehensive information about validators at the conclusion of each epoch. This ensures that the indexer captures and indexes the most up-to-date data about validators, eliminating the issue of incomplete information.
  2. Added functionality to include information about the last block of the epoch in the indexing process. This addition ensures that the server maintains a complete record of validator information, providing a comprehensive view of the epoch.

@kobayurii kobayurii force-pushed the fix_validators branch 2 times, most recently from 5752274 to 6f87e5d Compare January 18, 2024 21:14
@kobayurii kobayurii self-assigned this Jan 18, 2024
@kobayurii kobayurii requested a review from khorolets January 18, 2024 21:24
@kobayurii kobayurii marked this pull request as ready for review January 18, 2024 21:24
@kobayurii
Copy link
Member Author

#159

@kobayurii kobayurii merged commit 0d95a5b into main Jan 19, 2024
3 checks passed
@kobayurii kobayurii deleted the fix_validators branch January 19, 2024 18:12
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