Skip to content

Commit 5d21364

Browse files
committed
Merge branch '2.19'
2 parents 507ea4a + d967e8b commit 5d21364

File tree

2 files changed

+6
-6
lines changed

2 files changed

+6
-6
lines changed

avro/src/main/java/tools/jackson/dataformat/avro/schema/StringVisitor.java

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -11,7 +11,6 @@
1111
import tools.jackson.databind.introspect.AnnotatedClass;
1212
import tools.jackson.databind.jsonFormatVisitors.JsonStringFormatVisitor;
1313
import tools.jackson.databind.jsonFormatVisitors.JsonValueFormat;
14-
import tools.jackson.databind.type.TypeFactory;
1514

1615
public class StringVisitor extends JsonStringFormatVisitor.Base
1716
implements SchemaBuilder
@@ -38,7 +37,8 @@ public void enumTypes(Set<String> enums) {
3837
public Schema builtAvroSchema() {
3938
// Unlike Jackson, Avro treats characters as an int with the java.lang.Character class type.
4039
if (_type.hasRawClass(char.class) || _type.hasRawClass(Character.class)) {
41-
return AvroSchemaHelper.numericAvroSchema(NumberType.INT, TypeFactory.defaultInstance().constructType(Character.class));
40+
// should we construct JavaType for `Character.class` in case of primitive or... ?
41+
return AvroSchemaHelper.numericAvroSchema(NumberType.INT, _type);
4242
}
4343
// [dataformats-binary#179]: need special help with UUIDs, to coerce into Binary
4444
// (could actually be

avro/src/test/java/tools/jackson/dataformat/avro/schema/DateTimeVisitor_builtAvroSchemaTest.java

Lines changed: 4 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -26,8 +26,6 @@
2626
@RunWith(Parameterized.class)
2727
public class DateTimeVisitor_builtAvroSchemaTest {
2828

29-
private static final TypeFactory TYPE_FACTORY = TypeFactory.defaultInstance();
30-
3129
@Parameter(0)
3230
public Class<?> testClass;
3331

@@ -73,14 +71,16 @@ public static Collection<?> testData() {
7371
LocalTime.class,
7472
JsonParser.NumberType.INT,
7573
Schema.Type.INT,
76-
"time-millis"},
74+
"time-millis"
75+
}
7776
});
7877
}
7978

8079
@Test
8180
public void builtAvroSchemaTest() {
8281
// GIVEN
83-
DateTimeVisitor dateTimeVisitor = new DateTimeVisitor(TYPE_FACTORY.constructSimpleType(testClass, null));
82+
final TypeFactory tf = TypeFactory.createDefaultInstance();
83+
DateTimeVisitor dateTimeVisitor = new DateTimeVisitor(tf.constructSimpleType(testClass, null));
8484
dateTimeVisitor.numberType(givenNumberType);
8585

8686
// WHEN

0 commit comments

Comments
 (0)