avro-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Hudson (JIRA)" <j...@apache.org>
Subject [jira] [Commented] (AVRO-1692) Allow more than one logical type for a Java class
Date Tue, 14 Jul 2015 15:38:05 GMT

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

Hudson commented on AVRO-1692:

SUCCESS: Integrated in AvroJava #532 (See [https://builds.apache.org/job/AvroJava/532/])
AVRO-1692. Allow more than one logical type for a Java class. Contributed by Ryan Blue. (tomwhite:
rev 1690939)
* /avro/trunk/CHANGES.txt
* /avro/trunk/lang/java/avro/src/main/java/org/apache/avro/generic/GenericData.java
* /avro/trunk/lang/java/avro/src/main/java/org/apache/avro/generic/GenericDatumWriter.java
* /avro/trunk/lang/java/avro/src/main/java/org/apache/avro/reflect/ReflectData.java
* /avro/trunk/lang/java/avro/src/main/java/org/apache/avro/reflect/ReflectDatumReader.java
* /avro/trunk/lang/java/avro/src/test/java/org/apache/avro/TestCircularReferences.java

> Allow more than one logical type for a Java class
> -------------------------------------------------
>                 Key: AVRO-1692
>                 URL: https://issues.apache.org/jira/browse/AVRO-1692
>             Project: Avro
>          Issue Type: Task
>          Components: java
>            Reporter: Ryan Blue
>            Assignee: Ryan Blue
>             Fix For: 1.8.0
>         Attachments: AVRO-1692.1.patch, AVRO-1692.2.patch
> The initial logical types implementation assumed that there would only be one logical
type per concrete class and kept a lookup table from class to conversion. There are cases
where this assumption doesn't work. For example, both time-millis and time-micros can be represented
by the same class, LocalTime, and the write path should use the correct logical type's conversion
depending on the schema.

This message was sent by Atlassian JIRA

View raw message