Skip to content

Introduce RSocketInteractionModel #3110

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged

Conversation

artembilan
Copy link
Member

  • Deprecate RSocketOutboundGateway.Command in favor of newly introduced
    top level RSocketInteractionModel
  • Deprecate setter, DSL and XML configurations for the deprecated
    RSocketOutboundGateway.Command in favor of newly introduced configurators
    for the mentioned RSocketInteractionModel
  • Add IntegrationRSocketEndpoint.getInteractionModels() contract for
    inbound endpoints.
    This way we can restrict mapped endpoints to the particular interaction
    model(s)
  • Add DSL and XML configuration for inbound interaction model option
  • Document changes

@artembilan artembilan added this to the 5.2.2 milestone Nov 25, 2019
* Deprecate `RSocketOutboundGateway.Command` in favor of newly introduced
top level `RSocketInteractionModel`
* Deprecate setter, DSL and XML configurations for the deprecated
`RSocketOutboundGateway.Command` in favor of newly introduced configurators
for the mentioned `RSocketInteractionModel`
* Add `IntegrationRSocketEndpoint.getInteractionModels()` contract for
inbound endpoints.
This way we can restrict mapped endpoints to the particular interaction
model(s)
* Add DSL and XML configuration for inbound interaction model option
* Document changes
* Fix consistency for `RetrieveSpec` in the `RSocketOutboundGateway`
and `rsocket.adoc`
@artembilan artembilan force-pushed the RSocket_InteractionModel branch from ab5a5b5 to 28e4c22 Compare December 2, 2019 18:51
@artembilan
Copy link
Member Author

Rebased to master with more code clean up and consistency for a RSocketRequester.RetrieveSpec introduced recently in SF.

@garyrussell , any chances to have this reviewed and merged for upcoming release this Wednesday?

Thanks

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.

LGTM; just some javadoc polishing needed.

}

/**
* Configure a SpEL expression to evaluate a {@link RSocketInteractionModel}
Copy link
Contributor

Choose a reason for hiding this comment

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

s/a/an/

for the RSocket request...

(all occurrences)

Copy link
Member Author

@artembilan artembilan Dec 2, 2019

Choose a reason for hiding this comment

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

On it. Thanks

@garyrussell garyrussell merged commit 42b812b into spring-projects:master Dec 2, 2019
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