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

Philip Zeyliger updated AVRO-142:
---------------------------------

    Attachment: AVRO-142.patch.v3.txt

Whoops; dropped a file in the commit.  ant test passes now.

> Removing unused fields, constructors, and imports
> -------------------------------------------------
>
>                 Key: AVRO-142
>                 URL: https://issues.apache.org/jira/browse/AVRO-142
>             Project: Avro
>          Issue Type: Improvement
>          Components: java
>            Reporter: Philip Zeyliger
>            Assignee: Philip Zeyliger
>            Priority: Minor
>         Attachments: AVRO-142.patch.text, AVRO-142.patch.v2.txt, 
> AVRO-142.patch.v3.txt
>
>
> The patch to follow removes some unused fields.  I've been setting up Eclipse 
> to work on Avro (patch forthcoming), and Eclipse is perhaps unreasonably 
> whiny about these things, but there were so few, that it seemed harmless to 
> fix them.
> DataFileReader had a "count" field, which was never read (only assigned), so 
> I've deleted it.
> Several classes had LOG fields that were never used; I deleted them.
> TestCompare had a handful of unused imports; just deleted them.

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