Skip to content

Route transaction commits through configured brokers - #26

Merged
Raezil merged 6 commits into
mainfrom
fix/broker-backed-transaction-commit
Oct 4, 2026
Merged

Raezil merged 6 commits into
mainfrom
fix/broker-backed-transaction-commit

Conversation

@Raezil

@Raezil Raezil commented Oct 4, 2026 •

Copy link
Copy Markdown
Member

Summary

  • make Transaction.Commit provider-aware
  • publish buffered transaction events to Redis, RabbitMQ, or a custom configured provider
  • preserve synchronous regular-handler execution for local stores
  • on broker failure, retain only the failed/unpublished suffix so retries do not duplicate confirmed publishes
  • update README.MD, examples/README.md, and the website documentation page

Tests

Added regression coverage for:

  • provider-backed commits bypassing local handlers while preserving event order
  • retry behavior after a partial broker publish failure

Documentation

The README and website docs now explain that transactions buffer locally until Commit, then follow the parent EventStore transport:

  • no provider → execute regular handlers synchronously
  • configured provider → publish buffered events to the provider; handlers run later through Consume
  • partial broker failure → retry starts from the first unpublished event

Behavior

A transaction still buffers events locally until Commit. At commit time, the parent EventStore decides the execution path:

  • no provider → execute regular handlers synchronously, as before
  • configured provider → publish buffered events to the provider; handlers run later through Consume

@vercel

vercel Bot commented Oct 4, 2026 •

Copy link
Copy Markdown

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated
goeventbus Ready Ready Preview Oct 4, 2026 6:24pm UTC
goeventbus-website Canceled Canceled Oct 4, 2026 6:24pm UTC

@Raezil Raezil changed the title Fix/broker backed transaction commit Route transaction commits through configured brokers Oct 4, 2026
@Raezil Raezil self-assigned this Oct 4, 2026
@Raezil
Raezil merged commit b22f02b into main Oct 4, 2026
5 checks passed

This branch was successfully deployed

3 active (2 outdated) deployments
Preview — af905a2c Deployed Oct 4, 2026 by vercel[bot]
Preview – goeventbus — 038a77ae Deployed Oct 4, 2026 by vercel[bot]
Preview – goeventbus-website — 038a77ae Deployed Oct 4, 2026 by vercel[bot]
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.

1 participant