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

Hudson commented on AVRO-1544:
------------------------------

SUCCESS: Integrated in AvroJava #471 (See 
[https://builds.apache.org/job/AvroJava/471/])
AVRO-1544. Java: Fix GenericData#validate for unions with null.  Contributed by 
Matthew Hayes. (cutting: rev 1615764)
* /avro/trunk/CHANGES.txt
* 
/avro/trunk/lang/java/avro/src/main/java/org/apache/avro/generic/GenericData.java
* 
/avro/trunk/lang/java/avro/src/test/java/org/apache/avro/generic/TestGenericData.java


> Union of enum and null can result in NPE on validate if null is not first
> -------------------------------------------------------------------------
>
>                 Key: AVRO-1544
>                 URL: https://issues.apache.org/jira/browse/AVRO-1544
>             Project: Avro
>          Issue Type: Bug
>          Components: java
>    Affects Versions: 1.7.6
>            Reporter: Matthew Hayes
>            Assignee: Matthew Hayes
>             Fix For: 1.7.8
>
>         Attachments: AVRO-1544-v2.patch, AVRO-1544.patch
>
>
> The ENUM case in validate is missing a null check for datum:
> {code}
>     case ENUM:
>       return schema.getEnumSymbols().contains(datum.toString());
> {code}
> This leads to a surprising error where a NPE is thrown for a union of enum 
> and null when the enum is first in the union.  If null is first it works.  
> The fix is a simple.  I'm creating a patch that adds the fix and a unit test 
> for this case.



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

Reply via email to