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

indexer: add warnings when FN returns unexpected results #10813

Merged
merged 1 commit into from
Apr 12, 2023

Conversation

gegaowp
Copy link
Contributor

@gegaowp gegaowp commented Apr 12, 2023

Description

Unexpected responses from FN could block indexers from moving forward.

Test Plan

local run with checkpoint numbers and make sure that warnings pop

2023-04-12T17:10:00.913403Z  WARN sui_indexer::handlers::checkpoint_handler: Unexpected response from fullnode for checkpoints: Unexpected None value in SuiTransactionBlockFullResponse with error Transaction is None in SuiTransactionBlockFullResponse of digest TransactionDigest(JBBRdsd4UnrQ8haRYS2RoCkEJFFoUduSATZ78oBSdkhC).
2023-04-12T17:10:00.913435Z  WARN sui_indexer::handlers::checkpoint_handler: No checkpoints downloaded, retrying in next iteration ...
2023-04-12T17:10:00.944678Z  WARN sui_indexer::handlers::checkpoint_handler: Unexpected response from fullnode for object checkpoints: Unexpected None value in SuiTransactionBlockFullResponse with error Transaction is None in SuiTransactionBlockFullResponse of digest TransactionDigest(JBBRdsd4UnrQ8haRYS2RoCkEJFFoUduSATZ78oBSdkhC).
2023-04-12T17:10:00.944710Z  WARN sui_indexer::handlers::checkpoint_handler: No object checkpoints downloaded, retrying in next iteration ...

@vercel
Copy link

vercel bot commented Apr 12, 2023

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

4 Ignored Deployments
Name Status Preview Comments Updated (UTC)
explorer ⬜️ Ignored (Inspect) Apr 12, 2023 5:12pm
explorer-storybook ⬜️ Ignored (Inspect) Apr 12, 2023 5:12pm
sui-wallet-kit ⬜️ Ignored (Inspect) Apr 12, 2023 5:12pm
wallet-adapter ⬜️ Ignored (Inspect) Apr 12, 2023 5:12pm

@gegaowp gegaowp requested review from 666lcz, patrickkuo and wlmyng and removed request for 666lcz, patrickkuo and wlmyng April 12, 2023 16:01
@gegaowp gegaowp enabled auto-merge (squash) April 12, 2023 17:12
@gegaowp gegaowp changed the title indexer easy: add more step processing info indexer: add warnings when FN returns unexpected results Apr 12, 2023
@gegaowp gegaowp merged commit 6734857 into MystenLabs:main Apr 12, 2023
@gegaowp gegaowp deleted the writer-debug branch April 12, 2023 19:11
gegaowp added a commit to gegaowp/sui that referenced this pull request Apr 13, 2023
## Description 

Unexpected responses from FN could block indexers from moving forward.

## Test Plan 

local run with checkpoint numbers and make sure that warnings pop
```
2023-04-12T17:10:00.913403Z  WARN sui_indexer::handlers::checkpoint_handler: Unexpected response from fullnode for checkpoints: Unexpected None value in SuiTransactionBlockFullResponse with error Transaction is None in SuiTransactionBlockFullResponse of digest TransactionDigest(JBBRdsd4UnrQ8haRYS2RoCkEJFFoUduSATZ78oBSdkhC).
2023-04-12T17:10:00.913435Z  WARN sui_indexer::handlers::checkpoint_handler: No checkpoints downloaded, retrying in next iteration ...
2023-04-12T17:10:00.944678Z  WARN sui_indexer::handlers::checkpoint_handler: Unexpected response from fullnode for object checkpoints: Unexpected None value in SuiTransactionBlockFullResponse with error Transaction is None in SuiTransactionBlockFullResponse of digest TransactionDigest(JBBRdsd4UnrQ8haRYS2RoCkEJFFoUduSATZ78oBSdkhC).
2023-04-12T17:10:00.944710Z  WARN sui_indexer::handlers::checkpoint_handler: No object checkpoints downloaded, retrying in next iteration ...
```
gegaowp added a commit that referenced this pull request Apr 13, 2023
#10813 
#10830 
#10844

---------

Co-authored-by: Patrick Kuo <patrickkuo@me.com>
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