mobility-rpc icon indicating copy to clipboard operation
mobility-rpc copied to clipboard

Using CompletableFuture would simplify the code and be more powerful for users

Open mikehearn opened this issue 9 years ago • 1 comments

It'd be great if there were async versions of execute. It looks easy to add: just replace FutureExecutionResponse with a real CompletableFuture, return it from the execute methods and then change the current blocking execute methods to simply call get() on them.

mikehearn avatar Feb 27 '16 13:02 mikehearn

Hi Mike, good suggestion, thanks!

I'm in the middle of some work on my other project CQEngine at the moment, so it might be a while before I get to this.

However Mobility-RPC is open to contributions. So if you'd like to submit a pull request which adds this feature, I would be happy to merge it, as your idea sounds good!

npgall avatar Feb 29 '16 23:02 npgall