Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

time.Time #187

Open
jweibel22 opened this issue Apr 17, 2022 · 1 comment
Open

time.Time #187

jweibel22 opened this issue Apr 17, 2022 · 1 comment

Comments

@jweibel22
Copy link

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)

@actgardner
Copy link
Owner

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.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants