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

[proxy] improve response times by ~2x #166

Draft
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

drunkwinter
Copy link
Collaborator

@drunkwinter drunkwinter commented Nov 20, 2022

Found out that Shorts where using this parameter and those requests where considerably faster than those without.

Found out that Shorts where using this parameter and those requests where considerably faster than those without.

I don't know what this parameter means, but I can imagine it tells the server that the response should be for Shorts and possibly skips some checks and omits some data which we don't use anyway.
@drunkwinter
Copy link
Collaborator Author

@zerodytrash How did the test go? I think it wasn't quite the 2x, but more like 1.5x from what I saw in the stats.

@sayedhafiz

This comment was marked as spam.

@drunkwinter drunkwinter marked this pull request as draft May 13, 2023 00:49
@drunkwinter drunkwinter removed the request for review from zerodytrash October 12, 2023 23:04
@drunkwinter drunkwinter marked this pull request as ready for review October 12, 2023 23:04
@drunkwinter drunkwinter marked this pull request as draft October 12, 2023 23:05
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.

None yet

2 participants