opwvhk commented on PR #3354:
URL: https://github.com/apache/avro/pull/3354#issuecomment-2965274460

   > I also ericData` class is correctly setup today with values preserved in 
their canonical type (and not logical type). I did try the change, however, and 
noticed that unit-tests like this one started failing - 
`TestGenericLogicalTypes.writeLocalTimestampMicros`
   
   You have a point there, although I believe it's more a lack of being able to 
ignore a logical type (the class API does not provide for this).
   
   Adjusting that would be too much for this PR though.
   
   


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]

Reply via email to