Skip to content

Remove MultiProcessor API#280

Merged
serbrech merged 5 commits into
mainfrom
codex/remove-multi-processor
Jun 9, 2026
Merged

Remove MultiProcessor API#280
serbrech merged 5 commits into
mainfrom
codex/remove-multi-processor

Conversation

@karenychen

Copy link
Copy Markdown
Collaborator

Summary

  • remove the unused NewMultiProcessor API and its ReceiverEx wrapper
  • restore Processor to a single receiver while preserving retry, MaxReceiveCount, metrics, and panic recovery behavior
  • remove MultiProcessor docs/examples/tests and keep single-processor panic recovery coverage

Verification

  • go test ./... (from v2)
  • go vet ./... (from v2)
  • git diff --check
  • e2e module compiled against local v2 via temporary modfile/replace

@karenychen karenychen marked this pull request as draft June 5, 2026 01:02
Comment thread v2/metrics/processor/types_test.go Outdated
Comment thread v2/processor.go Outdated
Comment thread v2/processor.go Outdated
@karenychen

Copy link
Copy Markdown
Collaborator Author

Code review

No issues found. Checked for bugs and AGENTS.md compliance.

@karenychen karenychen force-pushed the codex/remove-multi-processor branch from d9ebce7 to f383f31 Compare June 8, 2026 22:08
@karenychen karenychen marked this pull request as ready for review June 9, 2026 00:35
Comment thread v2/processor.go Outdated
Comment thread v2/processor.go Outdated
Comment thread v2/processor.go Outdated
Comment thread v2/processor.go Outdated
@serbrech serbrech merged commit 3d02adc into main Jun 9, 2026
6 checks passed
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