Skip to content

Conversation

artembilan
Copy link
Member

Fixes: #10090

  • Rework amqp-1.0.adoc code snippets to tabs to demonstrate AMQP 1.0 channel adapters with different supported DSLs

@artembilan artembilan requested a review from cppwfs October 6, 2025 20:30
Copy link
Contributor

@cppwfs cppwfs left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks great!
2 nitpicks.

Thanks for the docs and tests!

Copy link
Contributor

@cppwfs cppwfs left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good!

Just my usual nitpicks. 😄 If you agree with the nitpicks I can fix on merge.

@artembilan
Copy link
Member Author

Just my usual nitpicks.

That's not nit-pick.
This is great review.

Thanks. On it.

Fixes: spring-projects#10090

* Rework `amqp-1.0.adoc` code snippets to tabs to demonstrate
AMQP 1.0 channel adapters with different supported DSLs
* Remove from `AmqpClientInboundGateway` wrong delegate for the `taskScheduler`
since gateway does not do batches
* Remove `taskScheduler` property from the `AmqpClientInboundGatewaySpec`
* Correct `AmqpClientMessageHandler` Javadoc about `SmartMessageConverter`
Copy link
Contributor

@cppwfs cppwfs left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

Thanks for the edits!!!

@cppwfs cppwfs merged commit 9f86108 into spring-projects:main Oct 8, 2025
3 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.

Implement Channel Adapters for AMQP 1.0 based on spring-rabbitmq-client
2 participants