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

Upgrade indexer #4552

Merged
merged 6 commits into from
Oct 5, 2022
Merged

Upgrade indexer #4552

merged 6 commits into from
Oct 5, 2022

Conversation

miiu96
Copy link
Contributor

@miiu96 miiu96 commented Oct 4, 2022

Description of the reasoning behind the pull request

Proposed Changes

  • A new indexer version will be used in order to fix the problem of how the ESDTWipe log is processed on the indexer side

Testing procedure

  • Create an NFT and do an ESDTWipe operation, after that, this NFT should no longer appear in the accountsesdt index

go.mod Outdated
@@ -8,7 +8,7 @@ require (
github.com/ElrondNetwork/arwen-wasm-vm/v1_4 v1.4.58
github.com/ElrondNetwork/concurrent-map v0.1.3
github.com/ElrondNetwork/covalent-indexer-go v1.0.6
github.com/ElrondNetwork/elastic-indexer-go v1.2.41
github.com/ElrondNetwork/elastic-indexer-go v1.2.42-0.20221004084418-690ed65c9bc9
Copy link
Contributor

Choose a reason for hiding this comment

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

proper release

Copy link
Contributor Author

Choose a reason for hiding this comment

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

added

@codecov-commenter
Copy link

codecov-commenter commented Oct 4, 2022

Codecov Report

Base: 73.95% // Head: 73.95% // Decreases project coverage by -0.00% ⚠️

Coverage data is based on head (bc345e3) compared to base (3c1a1d6).
Patch has no changes to coverable lines.

Additional details and impacted files
@@            Coverage Diff             @@
##           master    #4552      +/-   ##
==========================================
- Coverage   73.95%   73.95%   -0.01%     
==========================================
  Files         681      681              
  Lines       87162    87162              
==========================================
- Hits        64462    64459       -3     
- Misses      17883    17886       +3     
  Partials     4817     4817              
Impacted Files Coverage Δ
...t/processor/peerAuthenticationRequestsProcessor.go 92.30% <0.00%> (-2.57%) ⬇️

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

☔ View full report at Codecov.
📢 Do you have feedback about the report comment? Let us know in this issue.

@miiu96 miiu96 self-assigned this Oct 4, 2022
Copy link
Contributor

@gabi-vuls gabi-vuls left a comment

Choose a reason for hiding this comment

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

System test passed.

@iulianpascalau iulianpascalau merged commit cc415cc into master Oct 5, 2022
@iulianpascalau iulianpascalau deleted the master-with-latest-indexer branch October 5, 2022 06:48
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants