Skip to content

Indexer Refactoring Strategy Documentation#1182

Open
bogdanfazakas wants to merge 6 commits intomainfrom
docs/indexer-events
Open

Indexer Refactoring Strategy Documentation#1182
bogdanfazakas wants to merge 6 commits intomainfrom
docs/indexer-events

Conversation

@bogdanfazakas
Copy link
Member

Fixes #1173 .

Changes proposed in this PR:

  • Indexer refactoring strategy document with architectural context and improved event handling proposals.

@bogdanfazakas bogdanfazakas marked this pull request as ready for review February 1, 2026 20:27
@bogdanfazakas
Copy link
Member Author

/run-security-scan

Copy link
Member

@alexcos20 alexcos20 left a comment

Choose a reason for hiding this comment

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

AI automated code review (Gemini 3).

Overall risk: low

Summary:
This pull request introduces a comprehensive documentation outlining a refactoring strategy for the Ocean Node Indexer focusing on enhanced event monitoring, error handling, and overall resilience. The document proposes moving to a queue-based system, introducing a new ddo_logs index for better tracking, and improving retry mechanisms. The proposed changes are well-articulated and address critical pain points in the current indexer architecture.

Comments:
• [ERROR][style] There's a typo in the link to the existing architecture document. Arhitecture.md should be Architecture.md.
• [WARNING][other] The status mentions Jan 27, 2026. Given the current date, this seems to be a date in the future. Could this be a typo and should be Jan 27, 2024? If it's intentionally a future date, it might be good to clarify that this is a forward-looking strategy proposal for that date.
• [INFO][style] For consistency with did and eventType, consider adding // Indexed to chainId in the DdoLog interface, as it's stated as 'Indexed' in the description for query capabilities.
• [INFO][style] In section 1, EVENT_ERROR_PROCESSING_QUEUE is described. In this section, EVENT_PROCESSING_QUEUE (primary) is listed. While contextually related, it might be beneficial to explicitly state if EVENT_ERROR_PROCESSING_QUEUE is a specific instance/use case of the broader EVENT_PROCESSING_QUEUE, or if they are intended to be distinct but related queues. A small clarification could improve understanding of the overall queue landscape.

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.

Prep indexer workflows docs

2 participants