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

fix: handle mempool error #222

Merged
merged 3 commits into from
Jul 10, 2024
Merged

fix: handle mempool error #222

merged 3 commits into from
Jul 10, 2024

Conversation

debendraoli
Copy link
Collaborator

@debendraoli debendraoli commented Jul 9, 2024

This pull request adds retry delay when there is a mempool error. The retey duration is set to 30 seconds assuming mempool well be freed by then and reties until succeeds.
This change aims to handle the case when the mempool is full and retry the transaction after a specified delay.

@debendraoli debendraoli self-assigned this Jul 9, 2024
@debendraoli debendraoli added bug Something isn't working Archway Relay labels Jul 9, 2024
@debendraoli debendraoli changed the title Add sleep when there is mempool error fix: handle mempool error Jul 9, 2024
relayer/chains/wasm/tx.go Outdated Show resolved Hide resolved
Copy link

codecov bot commented Jul 9, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 13.15%. Comparing base (478dea5) to head (12add3b).

Additional details and impacted files

Impacted file tree graph

@@            Coverage Diff             @@
##             main     #222      +/-   ##
==========================================
+ Coverage   11.47%   13.15%   +1.68%     
==========================================
  Files          85       70      -15     
  Lines       20409    17075    -3334     
==========================================
- Hits         2342     2247      -95     
+ Misses      17813    14586    -3227     
+ Partials      254      242      -12     

see 15 files with indirect coverage changes

bcsainju
bcsainju previously approved these changes Jul 9, 2024
@debendraoli debendraoli merged commit fef22fd into main Jul 10, 2024
7 of 8 checks passed
@debendraoli debendraoli deleted the fix/mempoolfull branch July 10, 2024 06:56
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working Relay
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants