[ 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)