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

adding change to reuse connections #122

Merged
merged 1 commit into from
Apr 5, 2024

Conversation

Matias-Barrios
Copy link
Contributor

No description provided.

@luca147
Copy link

luca147 commented Aug 5, 2021

Currently I'm facing this issue, the connection is never used

@Gabrieljev
Copy link

when is the release date for this issue?

@Matias-Barrios
Copy link
Contributor Author

@Gabrieljev I did not get your question. Do you want to know when we are merging this? If thats the question then the response would be asap. It depends on the authors replying to the issue.

@saravind-hotstar
Copy link

saravind-hotstar commented Apr 30, 2022

@Gowtham-Sai @Nydan Please can we merge this, this actually affects performance

@Nydan
Copy link
Contributor

Nydan commented May 9, 2022

Hi, @saravind-hotstar I am no longer has the access to merge this PR.
Please contact the other maintaner whom has access to merge.

@rajat25
Copy link

rajat25 commented Aug 25, 2022

@rShetty Can we please merge this PR?
This would fix a performance issue.

@gwthm-in
Copy link
Collaborator

gwthm-in commented Apr 5, 2024

@rajat25 My user id changed and hence didn't get the notification for this. I'll try to close this as soon as possible.

@gwthm-in gwthm-in merged commit 020e42a into gojek:master Apr 5, 2024
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.

8 participants