Skip to content

Conversation

@echo304
Copy link
Contributor

@echo304 echo304 commented Feb 17, 2023

Resolve #2377

DestinationTopic.Properties has isDltTopic() and isMainEndpoint() but not for retry topic. It adds isRetryTopic() method and adds tests to verify the correct behavior.

  • Add isRetryTopic() instance method to DestinationTopic.Properties

sangboak.lee added 3 commits February 17, 2023 23:49
…perties#isRetryTopic() method

Resolve spring-projects#2377

`DestinationTopic.Properties` has `isDltTopic()` and `isMainEndpoint()` but not for retry topic.
It adds `isRetryTopic()` method and adds tests to verify the correct behavior.

* Add `isRetryTopic()` instance method to `DestinationTopic.Properties`
…pic() method

Resolve spring-projects#2377

`DestinationTopic.Properties` has `isDltTopic()` and `isMainEndpoint()` but not for retry topic.
It adds `isRetryTopic()` method and adds tests to verify the correct behavior.

* Add `isRetryTopic()` instance method to `DestinationTopic.Properties`
Copy link
Contributor

@garyrussell garyrussell left a comment

Choose a reason for hiding this comment

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

Thanks for the contribution; please run ./gradlew clean check to run checkstyle rules.

@echo304 echo304 requested a review from garyrussell February 24, 2023 06:32
@garyrussell garyrussell merged commit 83711f8 into spring-projects:main Feb 27, 2023
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.

Feature Request: Provide DestinationTopic.Properties#isRetryTopic() method

2 participants