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

Luke Jackson commented on AVRO-2858:
------------------------------------

Some support has already been added to C++ in AVRO-1290, with "NaN", "Infinity" 
and "-Infinity" specified as string.

[https://github.com/apache/avro/blob/2d3b1fe7efd865639663ba785877182e7e038c45/lang/c%2B%2B/impl/parsing/JsonCodec.cc#L576-L604]

[https://github.com/apache/avro/blob/2d3b1fe7efd865639663ba785877182e7e038c45/lang/c%2B%2B/impl/json/JsonIO.cc#L53-L77]

There are also existing test cases in Java for the same string values:

[https://github.com/apache/avro/blob/3a098e8be4944216f811211c9e3240fd6f67a4ae/lang/java/avro/src/test/java/org/apache/avro/generic/TestGenericData.java#L400-L409]

[https://github.com/apache/avro/blob/335acc5770f7ad6bd6fdf4caa305373a79912d70/lang/java/ipc/src/test/java/org/apache/avro/TestSchema.java#L130-L148]

As such it probably makes sense to continue to use string values, rather than 
unquoted literals.

> SchemaBuilder.GenericDefault withDefault method does not handle Double.NaN 
> correctly
> ------------------------------------------------------------------------------------
>
>                 Key: AVRO-2858
>                 URL: https://issues.apache.org/jira/browse/AVRO-2858
>             Project: Apache Avro
>          Issue Type: Bug
>          Components: java
>    Affects Versions: 1.9.2
>            Reporter: Gordon Wang
>            Priority: Major
>         Attachments: AVRO-2858.patch
>
>
> Upgraded to Avro 1.9.2 and found when using the withDefault method in 
> SchemaBuilder.GenericDefault to set default value for a Double typed field, 
> it cannot set Double.NaN correctly, throwing out below error:
> {code}
>  Exception in thread "main" org.apache.avro.AvroTypeException: Invalid 
> default for field field: "NaN" not a "double"
>  at org.apache.avro.Schema.validateDefault(Schema.java:1540)
>  at org.apache.avro.Schema.access$500(Schema.java:87)
>  at org.apache.avro.Schema$Field.<init>(Schema.java:521)
>  at 
> org.apache.avro.SchemaBuilder$FieldBuilder.completeField(SchemaBuilder.java:2238)
>  at 
> org.apache.avro.SchemaBuilder$FieldBuilder.completeField(SchemaBuilder.java:2230)
>  at 
> org.apache.avro.SchemaBuilder$FieldBuilder.access$5300(SchemaBuilder.java:2148)
>  at 
> org.apache.avro.SchemaBuilder$GenericDefault.withDefault(SchemaBuilder.java:2545)
> {code}
> After debugging it seemed the *toJsonNode* method in SchemaBuilder is 
> returning a TextNode rather than a NUMBER typed one.
> The problem can be replicated using below sample code:
> {code}
> SchemaBuilder.FieldBuilder fieldBuilder = 
> SchemaBuilder.record("record").fields().name("field");
> Schema fieldSchema = Schema.create(Schema.Type.DOUBLE);
> SchemaBuilder.GenericDefault genericDefault = fieldBuilder.type(fieldSchema);
> genericDefault.withDefault(Double.NaN);
> // Exception thrown
> {code}
>  



--
This message was sent by Atlassian Jira
(v8.3.4#803005)

Reply via email to