Skip to content

Conversation

@Alann27
Copy link
Contributor

@Alann27 Alann27 commented Nov 26, 2025

Summary

Recreated index supplier logic to handle the messages and events that updates suppliers in parallel:

  • Removed unused supplier handlers and related code (e.g., _handleOldEventSupplierSlashed).
  • Consolidated supplier processing into indexSupplier.
  • Exported internal utility methods (parseAttribute, getAttributes, etc.) for broader use.
  • Deprecated event/message handling for suppliers now processed via indexSupplier.
  • Enhanced modularity for supplier-related functions.

Issue

Our previous logic was indexing every supplier update sequentially, one by one. This caused node runners updating their suppliers' stakes (i.e., updating their services) to take an extremely long time, causing the indexer to get stuck on that block and fall out of sync.

Type of change

Select one or more:

  • New feature, functionality or library
  • Bug fix
  • Code health or cleanup
  • Documentation
  • Other (specify)

Sanity Checklist

  • I have tested my changes using the available tooling
  • I have commented my code
  • I have performed a self-review of my own code; both comments & source code
  • I create and reference any new tickets, if applicable
  • I have left TODOs throughout the codebase, if applicable

@Alann27 Alann27 requested a review from jorgecuesta November 26, 2025 16:14
@Alann27 Alann27 self-assigned this Nov 26, 2025
@Alann27 Alann27 added the enhancement New feature or request label Nov 26, 2025
- Removed unused supplier handlers and related code (e.g., `_handleOldEventSupplierSlashed`).
- Consolidated supplier processing into `indexSupplier`.
- Exported internal utility methods (`parseAttribute`, `getAttributes`, etc.) for broader use.
- Deprecated event/message handling for suppliers now processed via `indexSupplier`.
- Enhanced modularity for supplier-related functions.
… stake and adding error handling for missing data.
@Alann27 Alann27 force-pushed the feature/update-index-suppliers branch from 158d505 to a0933fa Compare November 28, 2025 16:15
@jorgecuesta jorgecuesta merged commit 25467ac into main Nov 28, 2025
3 checks passed
@jorgecuesta jorgecuesta deleted the feature/update-index-suppliers branch November 28, 2025 16:49
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement New feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants