Skip to content

Use oj instead oj yajl #365

Closed
ccocchi opened this Issue Nov 22, 2012 · 5 comments

3 participants

@ccocchi
ccocchi commented Nov 22, 2012

Hi,

yajl-ruby is not the fastest JSON library anymore, oj is. And it comes with MultiJson support and a #dump method.

Since rabl passes at least half the time converting a hash to JSON, I think it would be a great performance improvement for you to enable it by default and replace it in your guides.

Cheers

@databyte
Collaborator

I've started using it as well. I can make the changes one day unless someone beats me to it.

@nesquena
Owner

Updated the production guide to mention oj, happy to change it in any other guides.

@databyte
Collaborator

Looks great to me. Closing issue. If @ccocchi wants to submit anything missing as a patch, go for it!

@databyte databyte closed this Nov 27, 2012
@nesquena
Owner

@databyte I wonder if we should just remove multi_json and depend on oj explicitly. Why would anyone not use oj? Is there any reason for us not to do that for 1.0?

@databyte
Collaborator

Nope, that works for me! I have Issue #334 earmarked for that.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Something went wrong with that request. Please try again.