Skip to content

[7.1.1 Cherry-pick] Fix access token expiry eviction in connection pool V2 - #4739

Merged
cheenamalhotra merged 1 commit into
release/7.1from
dev/automation/pr-4734-to-7.1.1
Sep 23, 2026
Merged

cheenamalhotra merged 1 commit into
release/7.1from
dev/automation/pr-4734-to-7.1.1

Conversation

@github-actions

Copy link
Copy Markdown

Cherry-pick of #4734 (7da9d39) into release/7.1.

Validate access token expiry before general checkout, preserving return and transaction-affinity behavior. Cover callback cache refresh and physical reuse across both pool implementations and sync/async opens.

Co-authored-by: Copilot App <223556219+Copilot@users.noreply.github.com>
@github-actions
github-actions Bot requested a review from a team as a code owner September 23, 2026 18:07
@github-actions github-actions Bot added this to the 7.1.1 milestone Sep 23, 2026
@github-project-automation github-project-automation Bot moved this to To triage in SqlClient Board Sep 23, 2026
@azure-pipelines

Copy link
Copy Markdown
Azure Pipelines:
There may be pipelines that require an authorized user to comment /azp run to run.

@cheenamalhotra cheenamalhotra moved this from To triage to In review in SqlClient Board Sep 23, 2026
@cheenamalhotra

Copy link
Copy Markdown
Member

/azp run

@azure-pipelines

Copy link
Copy Markdown
Azure Pipelines:
Successfully started running 2 pipeline(s).

@codecov

codecov Bot commented Sep 23, 2026

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 64.83%. Comparing base (3fded36) to head (653d431).
⚠️ Report is 3 commits behind head on release/7.1.

Additional details and impacted files
@@               Coverage Diff               @@
##           release/7.1    #4739      +/-   ##
===============================================
- Coverage        66.39%   64.83%   -1.56%     
===============================================
  Files              291      285       -6     
  Lines            45110    68089   +22979     
===============================================
+ Hits             29949    44148   +14199     
- Misses           15161    23941    +8780     
Flag Coverage Δ
CI-SqlClient ?
PR-SqlClient-Project 64.83% <100.00%> (?)

Flags with carried forward coverage won't be shown. Click here to find out more.

☔ View full report in Codecov by Harness.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@mdaigle
mdaigle enabled auto-merge (squash) September 23, 2026 21:33
@cheenamalhotra
cheenamalhotra merged commit e311b90 into release/7.1 Sep 23, 2026
210 checks passed
@cheenamalhotra
cheenamalhotra deleted the dev/automation/pr-4734-to-7.1.1 branch September 23, 2026 23:04
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Status: Done

Development

Successfully merging this pull request may close these issues.

2 participants