Skip to content

(2.12) Atomic batch: reject unsupported commit#7368

Merged
neilalexander merged 1 commit intomainfrom
maurice/batch-commit-unsupported
Sep 29, 2025
Merged

(2.12) Atomic batch: reject unsupported commit#7368
neilalexander merged 1 commit intomainfrom
maurice/batch-commit-unsupported

Conversation

@MauriceVanVeen
Copy link
Copy Markdown
Member

The server didn't explicitly reject anything other than Nats-Batch-Commit:1 as explained in the ADR. It needs to be explicitly rejected such that we can later extend this with Nats-Batch-Commit:eob.

Signed-off-by: Maurice van Veen github@mauricevanveen.com

@MauriceVanVeen MauriceVanVeen requested review from a team as code owners September 29, 2025 08:47
@MauriceVanVeen MauriceVanVeen marked this pull request as draft September 29, 2025 08:57
Signed-off-by: Maurice van Veen <github@mauricevanveen.com>
@MauriceVanVeen MauriceVanVeen force-pushed the maurice/batch-commit-unsupported branch from 9d23bc2 to e6f30f2 Compare September 29, 2025 09:12
@MauriceVanVeen MauriceVanVeen marked this pull request as ready for review September 29, 2025 09:25
Copy link
Copy Markdown
Member

@neilalexander neilalexander left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@neilalexander neilalexander merged commit af058b6 into main Sep 29, 2025
69 of 70 checks passed
@neilalexander neilalexander deleted the maurice/batch-commit-unsupported branch September 29, 2025 10:18
neilalexander added a commit that referenced this pull request Sep 29, 2025
Includes the following:

- #7337
- #7342
- #7344
- #7345
- #7347
- #7346
- #7348
- #7349
- #7350
- #7357
- #7356
- #7358
- #7359
- #7366
- #7367
- #7293
- #7368

Signed-off-by: Neil Twigg <neil@nats.io>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants