[ https://issues.apache.org/jira/browse/AVRO-2359?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16804008#comment-16804008 ]
Tim Roberts edited comment on AVRO-2359 at 3/28/19 3:06 PM: ------------------------------------------------------------ I've opened an early PR (see Issue Links) for on-going review. The work is not complete but I'd appreciate early feedback in order to determine if there are things that I've missed or overlooked. I believe that I've covered the 'core' of the changes needed to support logical types in C#, but all of the logical types outlined in the current specification need implementations (I currently only have "date"). was (Author: timjroberts): I've opened an early PR (see Issue Links) for on-going review. The work is not complete but I'd appreciate early feedback in order to determine if there are things that I've missed or overlooked. I believe that I've covered the 'core' of the changes needed to support logical types in C#, and the logical types outlined in the current specification need implementations. > Support Logical Types in C# > --------------------------- > > Key: AVRO-2359 > URL: https://issues.apache.org/jira/browse/AVRO-2359 > Project: Apache Avro > Issue Type: New Feature > Components: csharp > Reporter: Tim Roberts > Priority: Critical > > By not supporting Logical Types, the C# Avro implementation is severely > limited in terms of its interoperability in heterogeneous environments. While > the implementation will safely ignore logical type declarations in processed > schemas; at runtime the semantics of a "date" for example, will be lost when > receiving an Avro payload that was encoded by the Java platform using the > same schema. The C# implementation will never be able to retrieve the date > value from the encoded int. > I propose that the C# Avro implementation be extended to support the Logical > Types as defined by the current specification. I have also explored the > lang/csharp codebase and believe that I could produce a PR to support this. -- This message was sent by Atlassian JIRA (v7.6.3#76005)