search

JetBrains / Exposed

Kotlin SQL Framework
http://jetbrains.github.io/Exposed/
Apache License 2.0
8.1k stars 677 forks source link

Make Time function compatible with PostgreSQL #1979

Closed terminalnode closed 3 weeks ago

terminalnode commented 5 months ago

Related issue on YouTrack (opened by me): https://youtrack.jetbrains.com/issue/EXPOSED-276/Time-class-is-not-compatible-with-PostgreSQL

Not sure what the practice for testing these are, if something needs to be added. I can't seem to run the relevant tests locally without running out of memory, presumably due to all the databases containers being started.

terminalnode commented 5 months ago

While the related Date class does work in Postgres, it could also be changed to ::date for consistency. But I think that's more of a stylistic choice, so not including that unless asked to. :-)

terminalnode commented 3 weeks ago

Any progress on this?

joc-a commented 3 weeks ago

Fixed here