Skip to content

Confirmed reads default only for v2 connections#4419

Merged
bfops merged 1 commit intomasterfrom
bfops/revert-1.0-reads
Feb 24, 2026
Merged

Confirmed reads default only for v2 connections#4419
bfops merged 1 commit intomasterfrom
bfops/revert-1.0-reads

Conversation

@bfops
Copy link
Collaborator

@bfops bfops commented Feb 24, 2026

Description of Changes

Reducing scope of #4390 to only apply to V2 clients.

API and ABI breaking changes

I think this is an API change?

Expected complexity level and risk

1

Testing

@bfops bfops marked this pull request as ready for review February 24, 2026 04:51
@bfops bfops enabled auto-merge February 24, 2026 04:52
@bfops bfops added this pull request to the merge queue Feb 24, 2026
github-merge-queue bot pushed a commit that referenced this pull request Feb 24, 2026
# Description of Changes

Reducing scope of #4390
to only apply to V2 clients.

# API and ABI breaking changes

I think this is an API change?

# Expected complexity level and risk

1

# Testing

<!-- Describe any testing you've done, and any testing you'd like your
reviewers to do,
so that you're confident that all the changes work as expected! -->

- [ ] <!-- maybe a test you want to do -->
- [ ] <!-- maybe a test you want a reviewer to do, so they can check it
off when they're satisfied. -->

Co-authored-by: Zeke Foppa <bfops@users.noreply.github.com>
@bfops bfops removed this pull request from the merge queue due to a manual request Feb 24, 2026
@bfops
Copy link
Collaborator Author

bfops commented Feb 24, 2026

We aligned in a meeting to force-merge this one.

@bfops bfops merged commit 2b85157 into master Feb 24, 2026
52 of 55 checks passed
@bfops bfops deleted the bfops/revert-1.0-reads branch February 25, 2026 18:29
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants