avro-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Doug Cutting (JIRA)" <j...@apache.org>
Subject [jira] Updated: (AVRO-726) Make GenericDatumReader/GenericDatumWriter data member protected so that it can be used by the derived classes
Date Mon, 07 Mar 2011 20:15:59 GMT

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

Doug Cutting updated AVRO-726:
------------------------------

       Resolution: Fixed
    Fix Version/s:     (was: 1.5.1)
                   1.5.0
         Assignee: Doug Cutting  (was: Scott Carey)
     Hadoop Flags: [Reviewed]
           Status: Resolved  (was: Patch Available)

I committed this.

> Make GenericDatumReader/GenericDatumWriter data member protected so that it can be used
by the derived classes
> --------------------------------------------------------------------------------------------------------------
>
>                 Key: AVRO-726
>                 URL: https://issues.apache.org/jira/browse/AVRO-726
>             Project: Avro
>          Issue Type: Improvement
>          Components: java
>    Affects Versions: 1.5.0
>            Reporter: Xiaolu Ye
>            Assignee: Doug Cutting
>             Fix For: 1.5.0
>
>         Attachments: AVRO-726.1.patch, AVRO-726.patch
>
>
> Currently, GenericDatumReader/GenericDatumWriter data members are private. Is it possible
to make them protected so that we could extend those classes and create our own special DatumReader/Writer?
The reason we want to do that is because we've created our own base SpecificRecordEx that
implements SpecificRecord and added put/get for primitive types. We now want to extend the
GenericDatumReader/GenericDatumWriter to use those primitive put/get functions to reduce box/unbox
for better performance. 
> Thanks,
> Xiaolu

--
This message is automatically generated by JIRA.
For more information on JIRA, see: http://www.atlassian.com/software/jira

Mime
View raw message