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

Hudson commented on AVRO-2836:
------------------------------

FAILURE: Integrated in Jenkins build AvroJava #890 (See 
[https://builds.apache.org/job/AvroJava/890/])
AVRO-2836 Generated java includes logical type conversions (#883) (github: 
[https://github.com/apache/avro/commit/ab83dcd6fee848cb0a82a9da328e114cc9e3568b])
* (add) 
lang/java/integration-test/codegen-test/src/test/resources/avro/nested_logical_types_union_fixed.avsc
* (edit) 
lang/java/integration-test/test-custom-conversions/src/main/java/org/apache/avro/codegentest/CustomDecimal.java
* (edit) 
lang/java/integration-test/codegen-test/src/test/java/org/apache/avro/codegentest/TestNestedLogicalTypes.java
* (edit) 
lang/java/compiler/src/main/java/org/apache/avro/compiler/specific/SpecificCompiler.java
* (edit) 
lang/java/integration-test/test-custom-conversions/src/main/java/org/apache/avro/codegentest/CustomDecimalConversion.java


> SpecificCompiler does not add DecimalConversion when logical type is a Fixed 
> type
> ---------------------------------------------------------------------------------
>
>                 Key: AVRO-2836
>                 URL: https://issues.apache.org/jira/browse/AVRO-2836
>             Project: Apache Avro
>          Issue Type: Bug
>          Components: java, logical types
>    Affects Versions: 1.9.2
>            Reporter: Matthew McMahon
>            Assignee: Matthew McMahon
>            Priority: Major
>             Fix For: 1.10.0
>
>         Attachments: AVRO-2836.patch
>
>
> I have updated to Avro 1.9.2 using the avro-maven-plugin to generate the 
> specific types.
> This is working nicely, except noticed the case of:
> {code:java}
> "fields": [
>     {
>       "name": "unionOfFixedDecimal",
>       "type": ["null", {
>         "namespace": "org.apache.avro.codegentest.testdata",
>         "name": "FixedInUnion",
>         "type": "fixed",
>         "size": 12,
>         "logicalType": "decimal",
>         "precision": 28,
>         "scale": 15
>       }]
>     }] {code}
> This is a fixed type that has a logical type of decimal.
> When the source is generated, the type is BigDecimal. However the 
> DecimalConversion is missing and then it breaks when used.
> It seems easy to workaround by manually adding the logical conversion before 
> using.
> However the fix seems to be in 
> SpecificCompiler#getClassNamesOfPrimitiveFields which is used by 
> #getUsedConversionClasses



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

Reply via email to