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.

Sign up for GitHub

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

Jump to bottom

[MEX-527] Bulk get methods #1535

Open
wants to merge 4 commits into
base: feat/memory-store
Choose a base branch
from
Open

[MEX-527] Bulk get methods #1535

wants to merge 4 commits into from
+432 −85

Conversation

mad2sm0key
Copy link
Contributor

@mad2sm0key mad2sm0key commented Dec 11, 2024 •
edited
Loading

Reasoning

  • optimize fetching of data from cache using redis MGET
  • these methods will be used by the memory store cron jobs

Proposed Changes

  • add bulk get methods (using getAllKeys util function) for various fields of pairs, farms, staking farms
  • update dataloaders where needed

How to test

  • N/A

@mad2sm0key
MEX-527: add farm bulk get methods
587aa9d
@mad2sm0key
MEX-527: add staking bulk get methods
392ff89
@mad2sm0key
MEX-527: add staking proxy bulk get methods
d9887c7
@mad2sm0key
MEX-527: add pair bulk get methods
32bcd82 
- bulk get methods for various fields
- update 'feesUSD' compute method signature to enable creating a dataloader for the field; update cache warmer + setter services; update usage in router service
- fix unit tests
@mad2sm0key mad2sm0key marked this pull request as ready for review December 12, 2024 12:34
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
1 participant
@mad2sm0key