gogen-avro icon indicating copy to clipboard operation
gogen-avro copied to clipboard

time.Time

Open jweibel22 opened this issue 2 years ago • 1 comments

Currently there's no support for the time.Time go type. One option would be to extend your library with support for a specifically named logical-type, e.g. "date-time", another option would be to add support for logical-types in general (where clients can provide custom type converters). What are your thoughts on this matter? Are logical types even being considered? (There's another very old issue in here about logical types in general)

jweibel22 avatar Apr 17 '22 06:04 jweibel22

Hi @jweibel22 - adding support for the time and duration logical types is something I would consider, because there's corresponding stdlib Go types for them. I would be open to a PR for this, I'm not sure when I'll have the bandwidth.

In the past users had asked about the decimal type, but AFAIK it would require adding a dependency on a specific implementation of arbitrary precision decimals.

actgardner avatar Apr 17 '22 14:04 actgardner