Skip to content

Rows use integers as id not uuid for MVT compatibility

Tristram Gräbener requested to merge row_id_integer into main

On veut diffuser toutes les données géographiques sous forme de tuiles MVT, indépendemment de la carte.

Le problème c’est qu’on veut également identifier les objets (par exemple si on clique dessus). Dans les données MVT on ajoute donc un identifiant unique.

Malheureusement, cet identifiant doit être un entier dans les tuiles, alors que nous utilisons des UUID.

Cette MR propose de changer les ID de UUID à SERIAL.

Les alternatives sont :

  • avoir une autre colonne d’identifiant que pour les MVT
  • générer un identifiant à la volée

L’id original est pour l’instant conservé, si jamais on a fait une boulette et devra être supprimé dans un deuxième temps.

Tout le cache est invalidé pour éviter d’avoir des erreurs d’identifiants dans le HTML.

Les ID des rows ne sont jamais utilisés « publiquement » (genre lien de partage, contrairement aux cartes et aux couches). Il y a un risque si une personne a la page chargée qu’elle veuille faire une opération ça n’aboutisse pas car ce n’est plus le bon ID. Je propose qu’on invite la personne à recharger sa page. Gérer des routes de migration me semble inutilement complexe pour ce cas-ci.

Merge request reports