Skip to content

Pass channel Config to PublisherDecorator and SubscriberDecorator#3454

Open
ozangunalp wants to merge 1 commit into
smallrye:mainfrom
ozangunalp:issue-1969-decorator-config
Open

Pass channel Config to PublisherDecorator and SubscriberDecorator#3454
ozangunalp wants to merge 1 commit into
smallrye:mainfrom
ozangunalp:issue-1969-decorator-config

Conversation

@ozangunalp

Copy link
Copy Markdown
Collaborator

Allow decorators to access the connector channel configuration

Also refactors PausableChannelDecorator to read the config directly from the passed channelConfig instead of pre-registering configurations, making it stateless and removing DefaultPausableChannelConfiguration.

Resolves #1969

@ozangunalp ozangunalp force-pushed the issue-1969-decorator-config branch from a8e2b5b to 2d03fbe Compare June 22, 2026 15:02
@codecov

codecov Bot commented Jun 22, 2026

Copy link
Copy Markdown

Codecov Report

❌ Patch coverage is 90.19608% with 5 lines in your changes missing coverage. Please review.
✅ Project coverage is 77.76%. Comparing base (a96442f) to head (2d03fbe).
⚠️ Report is 1206 commits behind head on main.

Files with missing lines Patch % Lines
...allrye/reactive/messaging/SubscriberDecorator.java 0.00% 2 Missing ⚠️
...ye/reactive/messaging/providers/wiring/Wiring.java 80.00% 0 Missing and 2 partials ⚠️
...mallrye/reactive/messaging/PublisherDecorator.java 0.00% 1 Missing ⚠️
Additional details and impacted files

Impacted file tree graph

@@             Coverage Diff              @@
##               main    #3454      +/-   ##
============================================
+ Coverage     77.47%   77.76%   +0.29%     
- Complexity     3778     5594    +1816     
============================================
  Files           306      484     +178     
  Lines         12673    18713    +6040     
  Branches       1648     2305     +657     
============================================
+ Hits           9818    14553    +4735     
- Misses         2116     3022     +906     
- Partials        739     1138     +399     
Files with missing lines Coverage Δ
...active/messaging/PausableChannelConfiguration.java 0.00% <ø> (ø)
...ye/reactive/messaging/jms/fault/JmsDlqFailure.java 92.45% <100.00%> (ø)
...ve/messaging/kafka/fault/KafkaDeadLetterQueue.java 82.43% <100.00%> (-3.08%) ⬇️
.../messaging/kafka/fault/KafkaDelayedRetryTopic.java 84.75% <100.00%> (-0.51%) ⬇️
...reactive/messaging/providers/AbstractMediator.java 83.21% <100.00%> (+0.13%) ⬆️
.../providers/extension/PausableChannelDecorator.java 91.30% <100.00%> (ø)
...aging/providers/impl/ConfiguredChannelFactory.java 65.97% <100.00%> (-5.98%) ⬇️
...tive/messaging/providers/impl/ConnectorConfig.java 83.48% <100.00%> (+4.24%) ⬆️
...mallrye/reactive/messaging/PublisherDecorator.java 0.00% <0.00%> (ø)
...allrye/reactive/messaging/SubscriberDecorator.java 0.00% <0.00%> (ø)
... and 1 more

... and 277 files with indirect coverage changes

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

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.

Decorators should have access to connector config

1 participant