[AVRO-2548] StringType of "String" causes logicalType converters to be ignored for field

• Type: Bug
• Status: Resolved
• Priority: Major
• Resolution: Fixed
• Affects Version/s: 1.9.0
• Fix Version/s: 1.9.2
• Component/s: java, logical types
• Assignee: Ivan Greene
• Reporter: Chris Woodham
• Created: 05/Sep/19 10:28
• Updated: 31/Jan/20 13:30
• Resolved: 01/Dec/19 10:54

Description

I have the avro maven plugin configured like this:

            <plugin>
                <groupId>org.apache.avro</groupId>
                <artifactId>avro-maven-plugin</artifactId>
                <version>${avro.version}</version>
                <executions>
                    <execution>
                        <goals>
                            <goal>schema</goal>
                        </goals>
                        <configuration>
                            <stringType>String</stringType>
                            <createSetters>false</createSetters>
                            <enableDecimalLogicalType>true</enableDecimalLogicalType>
                            <fieldVisibility>private</fieldVisibility>
                            <customConversions>
                                <conversion>
                                    org.apache.avro.Conversions$UUIDConversion
                                </conversion>
                            </customConversions>
                        </configuration>
                    </execution>
                </executions>
            </plugin>

With the intention of using the provided UUIDConversion class on string fields set with the logicalType uuid. However, it seems like having stringType specified as String means the converter is ignored.

If I comment out the line <stringType>String</stringType> then the converter is used as expected and the java classes have UUID types for those fields, but obviously all normal strings are now CharSequence objects.
 

Issue Links

• is duplicated by: AVRO-2628 SpecificCompiler erases Logical Types when string type is set to StringType.String
• links to: GitHub Pull Request #655