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.

Sign up for GitHub

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

Jump to bottom

indexer: increase backoff max interval to 10 min #7393

Merged
merged 1 commit into from
Jan 13, 2023
Merged

indexer: increase backoff max interval to 10 min #7393

merged 1 commit into from
Jan 13, 2023

Conversation

gegaowp
Copy link
Contributor

@gegaowp gegaowp commented Jan 13, 2023

increase max interval from 1 min to 10 min b/c on devnet run after hours, the event handler can die b/c of RPC TIMEOUT

2023-01-13T14:52:38.755422Z  WARN sui_indexer::handlers::handler_orchestrator: Indexer event handler failed with error: FullNodeReadingError("Failed reading event page with cursor Some(EventID { tx_seq: 385969, event_seq: 0 }) and error: RpcError(RequestTimeout)"), retrying...
2023-01-13T14:52:38.755446Z ERROR sui_indexer::handlers::handler_orchestrator: Indexer event handler failed after retrials with error: FullNodeReadingError("Failed reading event page with cursor Some(EventID { tx_seq: 385969, event_seq: 0 }) and error: RpcError(RequestTimeout)")

@vercel
Copy link

vercel bot commented Jan 13, 2023 •
edited
Loading

The latest updates on your projects. Learn more about Vercel for Git ↗︎

3 Ignored Deployments
Name Status Preview Comments Updated
explorer ⬜️ Ignored (Inspect) Jan 13, 2023 at 11:36PM (UTC)
explorer-storybook ⬜️ Ignored (Inspect) Jan 13, 2023 at 11:36PM (UTC)
wallet-adapter ⬜️ Ignored (Inspect) Jan 13, 2023 at 11:36PM (UTC)

@gegaowp gegaowp requested review from 666lcz and patrickkuo January 13, 2023 22:59
@gegaowp gegaowp force-pushed the indexer-backoff-interval branch from 8e11f93 to a727f4c Compare January 13, 2023 23:34
@gegaowp gegaowp requested a review from mwtian January 13, 2023 23:36
@gegaowp gegaowp enabled auto-merge (squash) January 13, 2023 23:44
@gegaowp gegaowp merged commit 62a5c1e into MystenLabs:main Jan 13, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@mwtian mwtian mwtian approved these changes

@666lcz 666lcz Awaiting requested review from 666lcz

@patrickkuo patrickkuo Awaiting requested review from patrickkuo

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@gegaowp @mwtian