search

rsocket / rsocket-rpc-java

Standard RSocket RPC Java Implementation
http://rsocket.io/
Apache License 2.0
172 stars 41 forks source link

Feature request: support for LoadBalancedRSocketMono #46

Open neilswingler opened 5 years ago

neilswingler commented 5 years ago

It would be cool if the client stub would accept a Mono\<Rsocket> in the constructor so a LoadBalancedRSocketMono could be passed in. Currently my solution is to create a new client stub instance for each request invocation which could have a noticeable memory consumption for a large number of concurrent requests.

robertroeser commented 5 years ago

We could wrap a the load balancer with an RSocket so that you don't have to create a new: https://gist.github.com/robertroeser/14cbadc21ef26e518fffe4ebcf0fe112

neilswingler commented 5 years ago

That will work without any new release I suppose. I guess you know best whether to use a Mono\<RSocket> or an RSocket wrapper.