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

Update Jackson version to post 2.x.x #71

Closed reistiago closed 10 years ago

reistiago commented 10 years ago

This would allow to use afterburner module that would improve serialization and deserialization performance

https://github.com/FasterXML/jackson-module-afterburner

eswdd commented 10 years ago

Locally I moved over to a post 2.x.x release this morning, which wasn't too painful. Unit tests pass fine. This requires me to do a bit of refactoring of where the ObjectMapper is created to enable the config to allow plugging in of this module. The default config will be to enable it, but the config will allow disabling it in case there are any probs.

reistiago commented 10 years ago

Cool, I'll keep an eye on this, to check for progress.

If you want, push a branch with your changes and I can try to help developing the remaining bits.

Thanks

reistiago commented 10 years ago

Thanks for the push. As far as I can quickly see this is complete and not needing me to do anything more.

Do you have any timeframe to release a version with this changes?

eswdd commented 10 years ago

I'm just waiting for the build to complete successfully - there's one test failure which looks related to this change so I'll fix that before marking as closed.

We don't have any timeframes for our next release as yet, but given we release approx every 3-4 months I would anticipate us doing a release in July or August.

eswdd commented 10 years ago

Hopefully that's the last of the tests which were wrong, so assuming the build now passes this will be closed.

eswdd commented 10 years ago

All tests passing.