Open graineri opened 3 years ago
I would love to give this one a try. My overall idea would be like this:
update_timestamp
column of type timestamptz
now()
in the insert into
statementThe challenge here is handling existing version tables. I would propose a change that leaves existing version tables alone. If a user wants this feature, we could provide documentation on migrating the version table.
pg_catalog
if a update_timestamp
column is presentupdate statement
that increments or decrements the versionupdate statement
, to set update_timestamp
to now()
What are your thoughts @jackc?
It's not something I've felt a particular need for, but if it were to be done I think it should be more extensive. That is, rather than simply record the time the last migration is run we should change the migration table to store a row for each migration with the migration number and the time it was run. This preserves more information and avoids the problem of what to do with the timestamp when rolling back.
As far as compatibility goes I suppose switching based on the structure of the version table is reasonable. Though I expect for there to be a tern v2 sometime this year (use pgx v5 and fs.FS) so maybe it would make sense to only make the change then (again if it is done at all).
Yes, great idea! And waiting for a version v2 seems reasonable to me. Thank you very much.
Adding such column can be useful to track when the migration was run.