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

[SERVICES-1824] improve escrow queries #1174

Merged
merged 5 commits into from
Sep 6, 2023

Conversation

claudiulataretu
Copy link
Collaborator

Reasoning

  • escrow queries ends calling SC via vm-queries or get all keys for every user

Proposed Changes

  • cache escrow SC keys to be used for all users
  • cache all addresses that has at least one permission in escrow SC
  • use cached escrow SC keys to get all receivers for a sender

How to test

escrowPermissions
escrowReceivers
  • queries should resolve in lesser time

- add caching for escrow SC keys
- add method to get addresses with permissions

Signed-off-by: Claudiu Lataretu <[email protected]>
- use cached addresses with permissions to check if the address has
  any permission

Signed-off-by: Claudiu Lataretu <[email protected]>
- use cached sc keys to get the escrow receivers for a sender

Signed-off-by: Claudiu Lataretu <[email protected]>
@claudiulataretu claudiulataretu merged commit 2d65d94 into development Sep 6, 2023
1 of 3 checks passed
@claudiulataretu claudiulataretu deleted the SERVICES-1824-improve-escrow-queries branch September 6, 2023 12:32
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 this pull request may close these issues.

3 participants