betfair / cougar

Cougar is a framework for making building network exposed service interfaces easy.
http://betfair.github.io/cougar
Apache License 2.0
27 stars 18 forks source link

Consider using netty #92

Open eswdd opened 9 years ago

eswdd commented 9 years ago

In theory could combine jetty and socket transports into a single unified transport..

Is supposedly faster than mina and is async all throughout.

richardqd commented 9 years ago

Could we drop hessian at the same time? Perf/correctness wins the day here!

On 3 December 2014 at 14:30, Simon Matic Langford notifications@github.com wrote:

In theory could combine jetty and socket transports into a single unified transport..

Is supposedly faster than mina and is async all throughout.

— Reply to this email directly or view it on GitHub https://github.com/betfair/cougar/issues/92.

eswdd commented 9 years ago

was planning to add a pluggable serialisation mechanism as part #61 which would in theory allow us to prefer non-hessian where available on both sides.

the challenge of switching netty for mina is maintaining compatibility during the switch - unless we support both for a period with a breaking change on the transport, which is less preferable i think.

I reckon we'd have to do something a bit cleverer around how we test the binary transport and compatibility before doing that - produce a test binary with each version which could be run on future versions for example.