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

store: write extensive tests that exorcize store/* #10298

Closed
4 tasks done
Tracked by #10360 ...
odeke-em opened this issue Oct 4, 2021 · 2 comments
Closed
4 tasks done
Tracked by #10360 ...

store: write extensive tests that exorcize store/* #10298

odeke-em opened this issue Oct 4, 2021 · 2 comments
Labels
T: Performance Performance improvements

Comments

@odeke-em
Copy link
Collaborator

odeke-em commented Oct 4, 2021

Summary

We need store/* at its best shape and to achieve that we need to exorcise all edges of it, finding what works, what conditions it trips under, how and when... When we finally have celestiaorg/smt as the store frontend it'll be very vital to ensure backend breakages don't occur silently but also to figure out how and what could be avoided. Almost similar to #10297

/cc @robert-zaremba @cuonglm in preparation for our Thursday meeting.


For Admin Use

  • Not duplicate issue
  • Appropriate labels applied
  • Appropriate contributors tagged
  • Contributor assigned/self-assigned
@odeke-em odeke-em self-assigned this Oct 4, 2021
@odeke-em odeke-em added the T: Performance Performance improvements label Oct 4, 2021
@robert-zaremba
Copy link
Collaborator

We can use simplified version of #10297 (comment) to test if after:

  • bunch of set / delete / we have the right data
  • after commits and rollbacks we still have the right data

@tac0turtle tac0turtle added the S:orijtech Squad: OrijTech label May 26, 2022
@tac0turtle tac0turtle moved this to 📝 Todo in Cosmos-SDK May 26, 2022
@tac0turtle tac0turtle removed the S:orijtech Squad: OrijTech label Oct 21, 2022
@tac0turtle
Copy link
Member

closing as facu is working from a different issue on this

Repository owner moved this from 📝 Todo to 👏 Done in Cosmos-SDK Oct 21, 2022
@tac0turtle tac0turtle removed this from Cosmos-SDK Nov 16, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
T: Performance Performance improvements
Projects
None yet
Development

No branches or pull requests

3 participants