[ 
https://issues.apache.org/jira/browse/AVRO-1260?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
 ]

Doug Cutting resolved AVRO-1260.
--------------------------------

       Resolution: Fixed
    Fix Version/s: 1.7.5
         Assignee: Martin Kleppmann

Looks great!  I tested the back compatibility by applying the patch then 
reverting the changes to random_data.rb and io.rb.  Tests still passed.

I committed this.  Thanks, Martin!
                
> Ruby: Performance improvements for parsing
> ------------------------------------------
>
>                 Key: AVRO-1260
>                 URL: https://issues.apache.org/jira/browse/AVRO-1260
>             Project: Avro
>          Issue Type: Improvement
>          Components: ruby
>    Affects Versions: 1.7.3
>            Reporter: Martin Kleppmann
>            Assignee: Martin Kleppmann
>             Fix For: 1.7.5
>
>         Attachments: AVRO-1260-2.patch, AVRO-1260.patch
>
>
> I've made a few performance improvements to the Ruby Avro decoder. In my 
> tests (decoding a complex record with many fields and nested records), I saw 
> a 40% speedup on MRI 1.8.7 and a 20% speedup on MRI 1.9.3. The improved code 
> also makes about half as many object allocations as before.
> Main changes are:
> * Using symbols instead of strings for the various schema types (there was 
> actually a FIXME in the code for this, now fixed)
> * Added a few memoizations to avoid unnecessarily recomputing values
> I've tried to keep the API unchanged — {{Avro::Schema#type}} still returns a 
> string, as users may be relying on the type ({{Avro::Schema#type_sym}} has 
> the symbolized version). So I don't think this patch introduces any 
> compatibility problems (as long as apps aren't relying on internals of the 
> Avro library).

--
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