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

Problem: get tx receipt api is too slow for indexing service #431

Closed
yihuang opened this issue Apr 19, 2022 · 0 comments · Fixed by #432
Closed

Problem: get tx receipt api is too slow for indexing service #431

yihuang opened this issue Apr 19, 2022 · 0 comments · Fixed by #432

Comments

@yihuang
Copy link
Collaborator

yihuang commented Apr 19, 2022

get tx receipt api is too slow for indexing service. one request could take more than 100ms, when there are dozens of transactions, total time could be seconds, exceeds the block time.
One solution is to add an api to fetch receipts in batch for a block.

@yihuang yihuang changed the title get tx receipt api is too slow for indexing service Problem: get tx receipt api is too slow for indexing service Apr 19, 2022
yihuang added a commit to yihuang/cronos that referenced this issue Apr 19, 2022
Closes: crypto-org-chain#431
Solution:
- add extension api: cronos_getTransactionReceiptsByBlock
yihuang added a commit to yihuang/cronos that referenced this issue Apr 19, 2022
Closes: crypto-org-chain#431
Solution:
- add extension api: cronos_getTransactionReceiptsByBlock
- add integration test
yihuang added a commit to yihuang/cronos that referenced this issue Apr 19, 2022
Closes: crypto-org-chain#431
Solution:
- add extension api: cronos_getTransactionReceiptsByBlock
- add integration test
yihuang added a commit to yihuang/cronos that referenced this issue Apr 19, 2022
Closes: crypto-org-chain#431
Solution:
- add extension api: cronos_getTransactionReceiptsByBlock
- add integration test
yihuang added a commit to yihuang/cronos that referenced this issue Apr 19, 2022
Closes: crypto-org-chain#431
Solution:
- add extension api: cronos_getTransactionReceiptsByBlock
- add integration test
yihuang added a commit to yihuang/cronos that referenced this issue Apr 20, 2022
Closes: crypto-org-chain#431
Solution:
- add extension api: cronos_getTransactionReceiptsByBlock
- add integration test
yihuang added a commit that referenced this issue Apr 26, 2022
* Problem: get tx receipt api is too slow for indexing service

Closes: #431
Solution:
- add extension api: cronos_getTransactionReceiptsByBlock
- add integration test

* cherry-pick to crypto-org-chain/ethermint fork

* support latest block identifier

* Update x/cronos/rpc/api.go
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 a pull request may close this issue.

1 participant