fix(connection): remove heartbeat check in load balanced mode #15089
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
cc @alexbevi
Fix #15042
Revert #14812
Summary
We've gotten a few bug reports that indicate connectivity issues with Mongoose 8.7, including #14971 etc. Most notably, we've confirmed that #15042 is caused by the heartbeat check we added in #14812: we should not use that heartbeat check if the topology is
LoadBalanced
because the MongoDB Node driver doesn't send heartbeats in that case, so for LoadBalanced topology Mongoose currently loses connectivity after 20 seconds. Given this and all the issues we've gotten specifically referencing connectivity issues in Mongoose 8.7.0, I think it is prudent to remove this check for now.Examples