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

Philip Zeyliger updated AVRO-166:
---------------------------------

    Attachment: AVRO-166.patch.txt

Attaching patch.

> Minor refactoring of Schema.java to better error reporting on bad schemas.
> --------------------------------------------------------------------------
>
>                 Key: AVRO-166
>                 URL: https://issues.apache.org/jira/browse/AVRO-166
>             Project: Avro
>          Issue Type: Improvement
>            Reporter: Philip Zeyliger
>         Attachments: AVRO-166.patch.txt
>
>
> Originally part of AVRO-152, this introduces getStringOptional and 
> getStringRequired helper methods, to facilitate throwing meaningful 
> exceptions when something's wrong with a schema.
> There aren't tests that bad schemas fail the validation in the expected ways, 
> but this did trigger a test which had a fixed field of size 0, which makes no 
> sense; that has been fixed.
> Where it was immediately adjacent, I've fixed brace-less if's according to 
> the style guide 
> (http://java.sun.com/docs/codeconv/html/CodeConventions.doc6.html#449).

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