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

(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
Collaborator

@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

Sign up for free to join this conversation on GitHub. Already have an account? Sign in 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