8000 GQL request handler redesign · Issue #652 · DevilXD/TwitchDropsMiner · GitHub
[go: up one dir, main page]
More Web Proxy on the site http://driver.im/
Skip to content

GQL request handler redesign #652

New issue
A932

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

Open
DevilXD opened this issue Dec 30, 2024 · 0 comments
Open

GQL request handler redesign #652

DevilXD opened this issue Dec 30, 2024 · 0 comments
Labels
Enhancement New feature or request

Comments

@DevilXD
Copy link
Owner
DevilXD commented Dec 30, 2024

Hello,

This is a self-note issue tracker for me, to remind myself to rework the GQL request handling code, so that composite requests are each handled separately. This is due to the fact that the requests can be made in batches up to 20 (even though I've seen the API handle up to 27 requests just fine, although that's probably only due to some kind of a response size limit), and if only one of them fails, all 20 are repeated again on a retry. This is not only really unnecessary, but also puts more strain on the Twitch servers for no real reason.

A proper implementation should including creating a request handler delegate for each request, then batching them together (most likely via timed polling, AKA "wait for X time before sending the request, just to see if the code adds more delegates in the mean time"), and then error handling and retry being separate from the operations that have finished successfully.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Enhancement New feature or request
Projects
None yet
Development

No branches or pull requests

1 participant
0