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
(v6.3.4#6332)

Mime
View raw message