Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

(POC) GODRIVER-3414 Complete pending reads on conn checkout #1937

Draft
wants to merge 5 commits into
base: v1
Choose a base branch
from

Conversation

prestonvasquez
Copy link
Member

@prestonvasquez prestonvasquez commented Feb 12, 2025

GODRIVER-3414

Summary

Continue reading server response when checking out connections that have experienced a network timeout, rather than as a background read.

Background & Motivation

The benefit to foreground reads is that it avoids opening new connections when all existing connections are stuck in the background reader.

@prestonvasquez prestonvasquez changed the base branch from master to v1 February 12, 2025 23:35
@prestonvasquez prestonvasquez changed the base branch from v1 to release/1.17 February 12, 2025 23:36
@prestonvasquez prestonvasquez changed the base branch from release/1.17 to v1 February 12, 2025 23:37
@mongodb-drivers-pr-bot mongodb-drivers-pr-bot bot added the priority-3-low Low Priority PR for Review label Feb 12, 2025
Copy link
Contributor

mongodb-drivers-pr-bot bot commented Feb 12, 2025

API Change Report

./event

compatible changes

ConnectionPendingReadFailed: added
ConnectionPendingReadStarted: added
ConnectionPendingReadSucceeded: added
PoolEvent.RemainingTime: added
PoolEvent.RequestID: added

./x/mongo/driver/topology

incompatible changes

BGReadCallback: removed
BGReadTimeout: removed

compatible changes

PendingReadTimeout: added

# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
priority-3-low Low Priority PR for Review
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant