Development #60507
sql : moderniser la création des index
0%
Description
Parce qu'en PostgreSQL 9.5+ il existe "IF NOT EXISTS" lors de la création d'un index, l'utiliser dans ceux qui sont créés, à l'instar de #60469
Fichiers
Demandes liées
Révisions associées
Historique
Mis à jour par Frédéric Péters il y a plus d'un an
- Assigné à mis à Frédéric Péters
(doit venir dans un cycle après #72859 pour pouvoir retirer la recherche des index existants dans do_formdef_indexes).
Mis à jour par Frédéric Péters il y a plus d'un an
- Lié à Development #72859: exécuter la mise à jour des indexes des formdef ajouté
Mis à jour par Frédéric Péters il y a plus d'un an
- Fichier 0001-sql-always-use-CREATE-INDEX-IF-NOT-EXISTS-60507.patch 0001-sql-always-use-CREATE-INDEX-IF-NOT-EXISTS-60507.patch ajouté
- Statut changé de Nouveau à Solution proposée
- Patch proposed changé de Non à Oui
Mis à jour par Pierre Ducroquet il y a plus d'un an
Je suis surpris, le patch ne supprime pas que des créations d'index, il y a aussi une création de primary key. Est-ce-qu'on suppose que la PK aura été créée partout, auquel cas, pourquoi s'embêter à garder toutes les vieilles migrations et n'enlever que celle-là ?
Mis à jour par Frédéric Péters il y a plus d'un an
Tu parles de : ?
- '''CREATE TABLE %s (id SERIAL, + '''CREATE TABLE %s (id SERIAL PRIMARY KEY,
C'est pour aller avec ce bout supprimé,
- if ('%s_pkey' % table_name) not in existing_indexes: - cur.execute('''ALTER TABLE %s ADD PRIMARY KEY (id)''' % table_name)
Et ce bout est supprimé pour ne plus avoir à récupérer existing_indexes.
Si tu parles d'autre chose je veux bien que tu précises.
Mis à jour par Frédéric Péters il y a plus d'un an
je veux
s/je veux/je veux bien/, bien sûr, corrigé.
Mis à jour par Robot Gitea il y a environ un an
Frédéric Péters (fpeters) a ouvert une pull request sur Gitea concernant cette demande :
- URL : https://gitea.entrouvert.org/entrouvert/wcs/pulls/42
- Titre : sql: always use CREATE INDEX IF NOT EXISTS (#60507)
- Modifications : https://gitea.entrouvert.org/entrouvert/wcs/pulls/42/files
Mis à jour par Robot Gitea il y a environ un an
- Statut changé de Solution proposée à Solution validée
Pierre Ducroquet (pducroquet) a approuvé une pull request sur Gitea concernant cette demande :
Mis à jour par Robot Gitea il y a environ un an
- Statut changé de Solution validée à Résolu (à déployer)
Frédéric Péters (fpeters) a mergé une pull request sur Gitea concernant cette demande :
- URL : https://gitea.entrouvert.org/entrouvert/wcs/pulls/42
- Titre : sql: always use CREATE INDEX IF NOT EXISTS (#60507)
- Modifications : https://gitea.entrouvert.org/entrouvert/wcs/pulls/42/files
Mis à jour par Transition automatique il y a environ un an
- Statut changé de Résolu (à déployer) à Solution déployée
sql: always use CREATE INDEX IF NOT EXISTS (#60507)