[ 
https://issues.apache.org/jira/browse/AVRO-1063?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13779465#comment-13779465
 ] 

Kevin Tham commented on AVRO-1063:
----------------------------------

Regarding the latter, it doesn't break compatibility. There's no functional 
change in Duke's patch other than lifting the trivial requirement on the yajl 
dependency. I hope this change gets into the next release. [~busbey] can you 
come back to give a quick review of Duke's patch? It basically the same as 
Paul's original pull request from Github from last year.
                
> Ruby client should use multi_json rather than being locked down to yajl
> -----------------------------------------------------------------------
>
>                 Key: AVRO-1063
>                 URL: https://issues.apache.org/jira/browse/AVRO-1063
>             Project: Avro
>          Issue Type: Improvement
>          Components: ruby
>            Reporter: Paul Dlug
>            Priority: Minor
>         Attachments: AVRO-1063.diff
>
>
> The avro ruby client uses yajl for JSON serialization which is just one of 
> many suitable JSON implementations for ruby. The multi_json gem provides a 
> wrapper for JSON serialization selecting the fastest library available (Oj is 
> now even faster than Yajl) and falling back to a pure ruby implementation 
> bundled with multi_json. Requiring yajl also precludes the ruby gem from 
> being used under jruby since it requires a C extension.

--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators
For more information on JIRA, see: http://www.atlassian.com/software/jira

Reply via email to