-
Notifications
You must be signed in to change notification settings - Fork 77
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
Support inserting PG::Interval
instances
#272
Comments
I've been thinking about this for the last little bit, and can't come up with any reasons not to. Do you want to send in a PR fix it? The only general thing to be aware of with postgres intervals it that it stores hours differently from days, and doesn't normalize by default. So you can have 2 days 31 hours, and 3 days 7 hours and it'll keep track that precision. But it'll know they're equal.
however for this issue, I don't think that trivia is relevant? But just wanted to bring it up in case. |
Does this matter on the crystal side that something like |
I don't see a problem in allowing PG::Interval on prepared statements. Whether that's by calling to_span or directly encoding the value is an implementation detail. I think is valuable to have the PG types as the safe options to handle the precision supported by the db, despite how a std-lib type might seem a 99% replacement. |
I ran into an interesting case where in order to insert data in a prepared statement, you need to use
Time::Span
, but in order to read it out, as noted in the README, you need to usePG::Interval
.Is there a reason
PG::Interval
shouldn't delegate to#to_span
to make it compatible in both directions?The text was updated successfully, but these errors were encountered: