Prefer JrJackson to JSON gem #112

Merged
merged 1 commit into from Apr 20, 2013

Conversation

Projects
None yet
2 participants
Contributor

jgwmaxwell commented Apr 20, 2013

Hi,

Really delighted to see JrJackson support in MultiJson, thanks to @guyboertje, @rwz and @sferik for all their hard work!

This pull request addresses the fact that I feel that JrJackson should be preferred to the JSON gem if it present, due to it being close to an order of magnitude more performant! Currently MultiJson will pluck out JSON as the backend in a rails install, even if JrJackson is in the Gemfile.

This change causes the opposite behaviour, delegating to the far more performant solution first, whereas all specs are still green.

Hope that people agree that this is a good idea, thanks for an awesome project!

John

sferik added a commit that referenced this pull request Apr 20, 2013

Merge pull request #112 from musicglue/master
Prefer JrJackson to JSON gem

@sferik sferik merged commit 035efe5 into intridea:master Apr 20, 2013

1 check passed

default The Travis build passed
Details
Member

sferik commented Apr 21, 2013

Thanks for the patch!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment