Project

General

Profile

Development #60507

sql : moderniser la création des index

Added by Thomas Noël about 1 year ago. Updated 9 days ago.

Status:
Solution déployée
Priority:
Normal
Target version:
-
Start date:
11 January 2022
Due date:
% Done:

0%

Estimated time:
Patch proposed:
Yes
Planning:
No

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


Files


Related issues

Related to w.c.s. - Development #72859: exécuter la mise à jour des indexes des formdefSolution déployée01 January 2023

Actions

Associated revisions

Revision db87f1bf (diff)
Added by Frédéric Péters 10 days ago

sql: always use CREATE INDEX IF NOT EXISTS (#60507)

History

#1

Updated by Frédéric Péters 27 days ago

  • Assignee set to 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).

#2

Updated by Frédéric Péters 27 days ago

#3

Updated by Frédéric Péters 27 days ago

#4

Updated by Pierre Ducroquet 24 days ago

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à ?

#5

Updated by Frédéric Péters 24 days ago

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.

#6

Updated by Frédéric Péters 24 days ago

je veux

s/je veux/je veux bien/, bien sûr, corrigé.

#7

Updated by Gitea (Bot) Gitea 10 days ago

Frédéric Péters (fpeters) a ouvert une pull request sur Gitea concernant cette demande :

#8

Updated by Gitea (Bot) Gitea 10 days ago

  • Status changed from Solution proposée to Solution validée

Pierre Ducroquet (pducroquet) a approuvé une pull request sur Gitea concernant cette demande :

#9

Updated by Gitea (Bot) Gitea 10 days ago

  • Status changed from Solution validée to Résolu (à déployer)

Frédéric Péters (fpeters) a mergé une pull request sur Gitea concernant cette demande :

#10

Updated by Transition automatique 9 days ago

  • Status changed from Résolu (à déployer) to Solution déployée

Also available in: Atom PDF