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

Limit the maximum number of concurrent (inflight) requests sent to Valkey through each connection of a Glide client #1253

Closed
4 tasks done
asafpamzn opened this issue Apr 9, 2024 · 2 comments
Closed
4 tasks done

Limit the maximum number of concurrent (inflight) requests sent to Valkey through each connection of a Glide client #1253

asafpamzn opened this issue Apr 9, 2024 · 2 comments
Assignees
@GilboaAWS
Labels
Core changes Used to label a PR as PR with significant changes that should trigger a full matrix tests.
Milestone
1.2

Comments

@asafpamzn
Copy link
Collaborator

asafpamzn commented Apr 9, 2024 •
edited by GilboaAWS
Loading

@asafpamzn asafpamzn added this to the GA milestone Apr 9, 2024
@asafpamzn asafpamzn modified the milestones: GA, node-GA Jun 26, 2024
@asafpamzn asafpamzn moved this to Todo misc in Valkey-GLIDE - internal Jul 9, 2024
@asafpamzn asafpamzn moved this from Todo misc to TODO bug fixes in Valkey-GLIDE - internal Jul 28, 2024
@asafpamzn asafpamzn assigned GilboaAWS and unassigned barshaul Jul 28, 2024
@asafpamzn asafpamzn moved this from TODO bug fixes to In Progress in Valkey-GLIDE - internal Aug 4, 2024
@ikolomi ikolomi modified the milestones: 1.1, 1.2 Sep 15, 2024
@ikolomi ikolomi mentioned this issue Sep 29, 2024
Merged
@asafpamzn asafpamzn removed this from the 1.2 milestone Sep 30, 2024
@GilboaAWS GilboaAWS changed the title Bound the multiplexer Q to prevent situation of out of memory at the rust side. Limit the maximum number of concurrent (inflight) requests sent to Valkey through each connection of a Glide client Sep 30, 2024
@asafpamzn asafpamzn added this to the 1.2 milestone Oct 1, 2024
This was referenced Oct 10, 2024
Closed
Closed
Merged
@ikolomi ikolomi added the Core changes Used to label a PR as PR with significant changes that should trigger a full matrix tests. label Oct 14, 2024
@GilboaAWS GilboaAWS mentioned this issue Oct 14, 2024
Merged
5 tasks
@avifenesh
Copy link
Member

It can be closed right?

@GilboaAWS
Copy link
Collaborator

We need to support GO as well. I'll discuss with @ikolomi if we create another issue or keep this one and target it to future release.

@github-project-automation github-project-automation bot moved this from In Progress to Done in Valkey-GLIDE - internal Oct 29, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@GilboaAWS GilboaAWS

Labels
Core changes Used to label a PR as PR with significant changes that should trigger a full matrix tests.
Projects
Valkey-GLIDE - internal
Status: Done
Milestone
1.2
Development

No branches or pull requests
5 participants
@avifenesh @barshaul @GilboaAWS @asafpamzn @ikolomi