Skip to content
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

Resources : filesize porté par l‘historique #3949

Open
wants to merge 3 commits into
base: master
Choose a base branch
from
Open

Conversation

ptitfred
Copy link
Contributor

@ptitfred ptitfred commented May 23, 2024

Le champ filesize n‘est plus lu de la base de données pour les ressources. La colonne n'est pas supprimée mais n'est plus connue de la couche de persistence.

L‘API est adaptée pour lire l‘historique (pour les resources GTFS en l‘état de la PR). Pour les autres types de ressources le filesize est null (pour l‘instant).

⚠️ Ceci signifie que certaines resources (voir commentaire ci-après) n'exposent plus de filesize dans l'API. Je ne suis pas sûr de la portée de ce changement.

See #2432.

Le champ `filesize` n‘est plus lu de la base de données pour les ressources.

L‘API est adaptée pour lire l‘historique (pour les resources GTFS en l‘état de
la PR). Pour les autres types de ressources le filesize est `null` (pour
l‘instant).

La colonne n'est pas supprimée mais n'est plus connue de la couche de
persistence.

See #2432.
@ptitfred
Copy link
Contributor Author

SELECT format, count(*)
FROM resource
WHERE
  format <> 'GTFS' AND
  filesize IS NOT NULL
GROUP BY 1
ORDER BY 2 DESC;
format count
geojson 153
csv 25
zip 12
pdf 6
ods 4
NeTEx 4
ssim 3
xlsx 2
xls 2
gpkg 2
gtfs-rt 2
0.2-20220529-1-.csv 1
png 1
json 1
shp 1

@ptitfred ptitfred changed the title Resources GTFS: filesize porté par l‘historique Resources : filesize porté par l‘historique May 23, 2024
@ptitfred ptitfred marked this pull request as ready for review May 29, 2024 12:42
@ptitfred ptitfred requested a review from a team as a code owner May 29, 2024 12:42
@ptitfred
Copy link
Contributor Author

En l'état, cette PR change le comportement de façon subtile et sans doute pas souhaitable : le filesize n'est plus renseigné pour les resources autres que GTFS.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant