[
https://issues.apache.org/jira/browse/AVRO-166?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
]
Doug Cutting updated AVRO-166:
------------------------------
Resolution: Fixed
Fix Version/s: 1.3.0
Status: Resolved (was: Patch Available)
I just committed this. I reverted some gratuitous style changes in the patch.
Thanks, Philip!
> 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
> Fix For: 1.3.0
>
> 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.
