TIMESTAMP WITH ZONE

[michaelkimchi]

Very minor thing but I wonder why we would want to store TIMESTAMP WITH ZONE, it is probably better not to store ZONE and the code is using Utc::now anyway.

https://github.com/jbr/async-sqlx-session/blob/main/src/pg.rs#L157

[michaelkimchi]

Hi, is this repo still active, I would like to do some other dependencies update/making changes, etc..

[jbr]

Yes, this repo is still active. I somehow was not watching it, so I did not see notifications. I don't know how that happened, but it's resolved now.

As far as TIMESTAMP WITH ZONE, I don't exactly recall why I chose that. If your sessions table is large enough that the storage size is relevant, I imagine there's additional db tuning we'd want to look at.

[michaelkimchi]

Thanks for replying @jbr

Yeah not critical, just that I try to avoid TIMESTAMP WITH ZONE wherever possible, it is all headache.

[jbr]

This makes sense. I'd be amenable to making this change, but we'd need to think through how to do migrations and track table versions, which was something I put off "for later"

[michaelkimchi]

Ah I was thinking just keep TIMESTAMP instead TIMESTAMP WITH ZONE so the changes should be minimal?