Produits Entr'ouvert » Chrono

(d'une note de Benjamin sur #44644)

Il y a des index d'exclusion en PostgreSQL : https://www.postgresql.org/docs/9.6/sql-createtable.html#SQL-CREATETABLE-EXCLUDE

Mais ça demanderait que end_datetime soit matérialisé dans la table (on ne peut pas demander à postgres de le calculer à la volée depuis meeting_type__duration comme on le fait dans le code).

Ça donnerait alors quelque chose comme ça :

EXCLUDE USING GIST (desk_id WITH =, tstzrange(start_datetime, end_datetime) WITH &&)

Il faut aussi que ça n'éclate pas les mises à jour de systèmes qui auraient de telles réservations, svp.

Support natif depuis django 3.0 des contraintes postgres https://docs.djangoproject.com/en/3.0/ref/contrib/postgres/constraints/ (pour l'anecdote).

Frédéric Péters a écrit :

Il faut aussi que ça n'éclate pas les mises à jour de systèmes qui auraient de telles réservations, svp.

• ajouter une colonne "order INTEGER" ayant pour valeur par défaut 0
• assigner des valeurs 1, 2.. etc.. pour les doublons
• utiliser comme contrainte d'unicité (desk_id, (start, end), order)

Une fois que les doublons ont disparu ou sont dans le passé, on peut virer order.

C'est le genre de truc plutôt galère à obtenir, il faudrait pouvoir tourner sans, genre comme ça s'est passé pour l'indexation full text (https://git.entrouvert.org/combo.git/tree/combo/apps/search/migrations/0007_french_fts.py).

-- Add technical columns
ALTER TABLE agendas_event ADD COLUMN end_datetime TIMESTAMP WITH TIME ZONE;
ALTER TABLE agendas_event ADD COLUMN ignore_reason CHARACTER VARYING(20);

J'ajouterais un préfixe particulier à ces colonnes, pour ne pas risquer un jour conflit avec ce qu'on pourrait ajouter comme attribut. Je n'étais pas sûr de la légalité mais postgresql autorise un _ en premier caractère, je ferais ça.