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

Scott Carey commented on AVRO-662:
----------------------------------

A quick review looks good.  I did not go into complete depth on 
SequenceFileReader though.

+1

The current Unit test covers 85% of the LOC in SequenceFileReader, but doesn't 
touch several data types and always hits the WRITABLE_SCHEMAS cache. 

> Java: Add InputFormat for SequenceFiles using Reflect API
> ---------------------------------------------------------
>
>                 Key: AVRO-662
>                 URL: https://issues.apache.org/jira/browse/AVRO-662
>             Project: Avro
>          Issue Type: New Feature
>          Components: java
>            Reporter: Doug Cutting
>            Assignee: Doug Cutting
>             Fix For: 1.4.1
>
>         Attachments: AVRO-662.patch
>
>
> It would be useful to be able to read SequenceFile-based data into an 
> Avro-based Java mapreduce program.  Once the reflect, specific and generic 
> representations are fully compatible (AVRO-638) then a RecordReader for 
> SequenceFiles could be added that uses Avro's reflect representation.  
> AvroOutputFormat could also be changed to accept such reflected data.

-- 
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.

Reply via email to