Skip to content

Add Inbound Channel Adapter for MQTT based on hivemq - #10989

Open
mjd507 wants to merge 3 commits into
spring-projects:mainfrom
mjd507:hivemq-mqtt
Open

Add Inbound Channel Adapter for MQTT based on hivemq#10989
mjd507 wants to merge 3 commits into
spring-projects:mainfrom
mjd507:hivemq-mqtt

Conversation

@mjd507

@mjd507 mjd507 commented May 6, 2026

Copy link
Copy Markdown
Contributor

Related to: #3102

To avoid constructor explosion, only MqttClientBuilder or MqttClientManager are designed as input params for building the adapters.

The MqttClientManager requires a MqttClientBuilder as well.

When build the underlying MqttClient, the config from the input MqttClientBuilder will be cloned to avoid global side effect.

@mjd507
mjd507 force-pushed the hivemq-mqtt branch 2 times, most recently from 8ed760f to 6d981ef Compare May 8, 2026 01:09
@mjd507

mjd507 commented May 12, 2026

Copy link
Copy Markdown
Contributor Author

here is the PR in my own repo for outbound v3/v5 adapters (mjd507#1).
I will modify and raise it once the inbound review completed.

Comment thread build.gradle Outdated
@mjd507
mjd507 marked this pull request as draft August 7, 2026 01:13
@artembilan
artembilan requested a review from cppwfs August 12, 2026 15:14
@artembilan

Copy link
Copy Markdown
Member

@cppwfs ,

this is no rush: for the next milestone or so.
Not for just planned release next week.

Thanks

@mjd507
mjd507 marked this pull request as ready for review August 15, 2026 22:39
Related to: spring-projects#3102

To avoid constructor explosion, only `MqttClientBuilder` or `MqttClientManager` are designed as input params for building the adapters.

The `MqttClientManager` requires a `MqttClientBuilder` as well.

When build the underlying `MqttClient`,
the config from the input `MqttClientBuilder` will be cloned to avoid global side effect.

Signed-off-by: Jiandong Ma <jiandong.ma.cn@gmail.com>
mjd507 added 2 commits August 18, 2026 16:37
Signed-off-by: Jiandong Ma <jiandong.ma.cn@gmail.com>
Signed-off-by: Jiandong Ma <jiandong.ma.cn@gmail.com>
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