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

Network Drive ftest fails if ping() is called too many times #2175

Closed
seanstory opened this issue Feb 15, 2024 · 0 comments · Fixed by #2182
Closed

Network Drive ftest fails if ping() is called too many times #2175

seanstory opened this issue Feb 15, 2024 · 0 comments · Fixed by #2182
Assignees
Labels
bug Something isn't working

Comments

@seanstory
Copy link
Member

Bug Description

When we tried to use ping() inside the backend loop for connectors, the Network Drive ftest began failing. The current theory is that this is because ping() creates a connection, and that too many calls to it exhausts the connection pool. We need to fix this so that we can work onhttps://github.com//issues/2173

To Reproduce

  1. revert Revert "Add logic to send a ping to connector's backend every loop (#2105)" #2148
  2. Run the network drive ftest

Expected behavior

We should be able to ping the source as much as we want, without it compromising the stability of the connector

@seanstory seanstory added the bug Something isn't working label Feb 15, 2024
@praveen-kukreja praveen-kukreja linked a pull request Mar 7, 2024 that will close this issue
12 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants