Skip to content

Add dsl for redis inbound/outbound channel adapter.#10905

Merged
artembilan merged 6 commits intospring-projects:mainfrom
mjd507:8108
Apr 1, 2026
Merged

Add dsl for redis inbound/outbound channel adapter.#10905
artembilan merged 6 commits intospring-projects:mainfrom
mjd507:8108

Conversation

@mjd507
Copy link
Copy Markdown
Contributor

@mjd507 mjd507 commented Mar 31, 2026

Related to: #8108

mjd507 added 2 commits March 31, 2026 13:48
Related to: spring-projects#8108

Signed-off-by: Jiandong Ma <jiandong.ma.cn@gmail.com>
Signed-off-by: Jiandong Ma <jiandong.ma.cn@gmail.com>
Copy link
Copy Markdown
Member

@artembilan artembilan left a comment

Choose a reason for hiding this comment

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

Thank you!

Final we got a crack on that issue!

Not too much, but still some review.

Signed-off-by: Jiandong Ma <jiandong.ma.cn@gmail.com>
@mjd507 mjd507 requested a review from artembilan March 31, 2026 22:31
Signed-off-by: Jiandong Ma <jiandong.ma.cn@gmail.com>
@mjd507 mjd507 changed the title Add dsl for redis inbound channel adapter. Add dsl for redis inbound/outbound channel adapter. Apr 1, 2026
Copy link
Copy Markdown
Member

@artembilan artembilan left a comment

Choose a reason for hiding this comment

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

Thank for the update !
I see you have gone a bit further than just inbound. That’s OK with me , however we still need reflect this change in docs: like mention with a short sentence in the ‘whats-new.adoc` and respective parts of target Redis chapter .

…rts in redis.adoc

Add topicExpression(String) and topicFunction(Function) in RedisOutboundChannelAdapterSpec.

Signed-off-by: Jiandong Ma <jiandong.ma.cn@gmail.com>
@mjd507 mjd507 requested a review from artembilan April 1, 2026 12:39
Signed-off-by: Jiandong Ma <jiandong.ma.cn@gmail.com>
Copy link
Copy Markdown
Member

@artembilan artembilan left a comment

Choose a reason for hiding this comment

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

Honestly, this is great and ready to be merged.
If you'd like to add more DSL for other Redis components, I would suggest to do that in other PR.

Thank you!

@artembilan artembilan added this to the 7.1.0-RC1 milestone Apr 1, 2026
@artembilan artembilan merged commit 445fbcb into spring-projects:main Apr 1, 2026
3 checks passed
@mjd507 mjd507 deleted the 8108 branch April 1, 2026 22:26
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants