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

Optimize subscription #10749

Closed
vikiival opened this issue Aug 5, 2024 · 4 comments · Fixed by #10764
Closed

Optimize subscription #10749

vikiival opened this issue Aug 5, 2024 · 4 comments · Fixed by #10764
Assignees
Labels
$ ~<50usd p1 preventing everyone from using app

Comments

@vikiival
Copy link
Member

vikiival commented Aug 5, 2024

Context

At this point we have it a very fancy behaviour of SubSquid API that it causes memory leak

Note

for humans - it means that app feels ultra slow

This is 99% caused by subscriptions and their incorrect usage

Task

  1. identify subscriptions in our app
  2. if possible optimize their size
  • if not possible we can always update schema
  1. if possible replace subscription with polling :)

Convos

Global

Screenshot 2024-08-05 at 14 55 39

Screenshot 2024-08-05 at 15 01 34

@vikiival vikiival added the p1 preventing everyone from using app label Aug 5, 2024
@vikiival
Copy link
Member Author

vikiival commented Aug 5, 2024

Assigning @preschian as he probably was implementing subscriptions last time, also help from anyone from @kodadot/internal-dev is welcomed :)

@vikiival vikiival added the $ ~<50usd label Aug 5, 2024
@preschian
Copy link
Member

Assigning @preschian as he probably was implementing subscriptions last time

Hah, me? I think it is false mention

one more reason for this: https://github.com/kodadot/ops-internal/discussions/1745

@vikiival
Copy link
Member Author

vikiival commented Aug 5, 2024

Hah, me? I think it is false mention

noted

one more reason for this: https://github.com/kodadot/ops-internal/discussions/1745

Won't help :), you will get same issue on other drops, chains etc

  • app is already slow

@preschian
Copy link
Member

Won't help :), you will get same issue on other drops, chains etc

Why?

For example, this PR works fine: #10713. unless, I just found some drops doesn't have supply functions

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
$ ~<50usd p1 preventing everyone from using app
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants