Skip to content

Use source transaction for exactly-once output - #802

Open
wbarnha wants to merge 3 commits into
masterfrom
copilot/fix-517-source-transaction-routing
Open

wbarnha wants to merge 3 commits into
masterfrom
copilot/fix-517-source-transaction-routing

Conversation

@wbarnha

@wbarnha wbarnha commented Oct 22, 2026

Copy link
Copy Markdown
Member

Summary

  • Route output sends made while processing an input event through that input partition's transaction, including when the sink topic is not subscribed and its partition metadata is unavailable.
  • Preserve destination-partition routing for sends without an active source event.
  • Add a regression test for an unassigned sink topic.

Addresses #517.

Validation

  • tests/unit/transport/test_consumer.py: 116 passed.

@codecov

codecov Bot commented Oct 22, 2026 •

Copy link
Copy Markdown

Codecov Report

❌ Patch coverage is 89.47368% with 2 lines in your changes missing coverage. Please review.
✅ Project coverage is 96.18%. Comparing base (3442a90) to head (d66b55a).

Files with missing lines Patch % Lines
faust/topics.py 85.71% 0 Missing and 1 partial ⚠️
faust/transport/consumer.py 91.66% 0 Missing and 1 partial ⚠️
Additional details and impacted files
@@            Coverage Diff             @@
##           master     #802      +/-   ##
==========================================
- Coverage   96.20%   96.18%   -0.02%     
==========================================
  Files         110      110              
  Lines       11791    11805      +14     
  Branches     1282     1287       +5     
==========================================
+ Hits        11343    11355      +12     
  Misses        351      351              
- Partials       97       99       +2     

☔ 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.

This branch has not been deployed

No deployments
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