You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@avro.apache.org by "Doug Cutting (JIRA)" <ji...@apache.org> on 2014/03/12 20:30:43 UTC

[jira] [Commented] (AVRO-1480) AvroTypeException should include fieldName as distinct field

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

Doug Cutting commented on AVRO-1480:
------------------------------------

Adding a subclass (AvroFieldNotFound or somesuch) seems better, if folks want to specially process this exception.

> AvroTypeException should include fieldName as distinct field
> ------------------------------------------------------------
>
>                 Key: AVRO-1480
>                 URL: https://issues.apache.org/jira/browse/AVRO-1480
>             Project: Avro
>          Issue Type: Improvement
>          Components: java
>    Affects Versions: 1.7.6
>            Reporter: Will Sargent
>            Priority: Minor
>
> Currently AvroTypeException is defined as 
> public class AvroTypeException extends AvroRuntimeException {
>   public AvroTypeException(String message) { super(message); }
>   public AvroTypeException(String message, Throwable cause) {
>     super(message, cause);
>   }
> }
> And when used in JSONDecoder, field name failures are defined like this:
> throw new AvroTypeException("Expected field name not found: " + fa.fname);
> This means that, in the event of an AvroTypeException, the only way to find which field has the error is to scan through the error message.
> If the AvroTypeException had a constructor (message, throwable, field) or a subclass which explicitly provided fields, then it would make error processing much easier.



--
This message was sent by Atlassian JIRA
(v6.2#6252)