[jira] Updated: (AVRO-166) Minor refactoring of Schema.java to better error reporting on bad schemas.

["Philip Zeyliger (JIRA)" <ji...@apache.org>]

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

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

    Assignee: Philip Zeyliger
      Status: Patch Available  (was: Open)

> 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
>            Assignee: 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.