search

ecolabdata / ecospheres

Portail des données de la transition écologique et de la cohésion des territoires
https://ecologie.data.gouv.fr
2 stars 0 forks source link

Type incorrect pour harvest->modified_at #410

Closed streino closed 1 month ago

streino commented 1 month ago

image

streino commented 1 month ago

Cf

Re les dates, on n'a pas fait de traitement particulier, mais ça ne devrait rien changer par rapport aux valeurs imbriquées du JSON qui étaient des strings ? Ou alors il y avait une détection de type automatique dans superset à ce moment là ? On peut s'arranger pour caster quelques colonnes en string dans la moulinette en datetime en tout cas.

https://ecosphres.slack.com/archives/C07KNCLLEJW/p1728394098621519?thread_ts=1728385718.382429&cid=C07KNCLLEJW

streino commented 1 month ago

Workaround :

image (2)

Mais ça serait mieux d'avoir un type propre en base.

streino commented 1 month ago

Temporairement corrigé avec un champ calculé, en attendant le fix en base.

streino commented 1 month ago

J'ai oublié la partie migration dans la PR... Fait à la main à l'instant : 

ALTER TABLE catalog ALTER COLUMN harvest__created_at TYPE TIMESTAMP USING harvest__created_at::timestamp;
ALTER TABLE catalog ALTER COLUMN harvest__modified_at TYPE TIMESTAMP USING harvest__modified_at::timestamp;
ALTER TABLE catalog ALTER COLUMN harvest__last_update TYPE TIMESTAMP USING harvest__last_update::timestamp;
ALTER TABLE catalog DROP COLUMN harvest;