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

fpeters · 2023-01-18T16:43:07+01:00

fpeters commented

2023-01-18 16:43:07 +01:00

No description provided.

fpeters added 1 commit 2023-01-18 16:43:07 +01:00

gitea-wip/wcs/pipeline/pr-main This commit looks good Details

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

pducroquet approved these changes 2023-01-18 17:21:02 +01:00

pducroquet left a comment

Ça me semble bien. La seule exception sur un tel changement serait la table sql_meta.
Vu qu'elle subit les migrations systématiquement (vu qu'elle contient le sql_level), ça serait néfaste de retirer le code qui fonctionne actuellement.

Ça me semble bien. La seule exception sur un tel changement serait la table sql_meta. Vu qu'elle subit les migrations systématiquement (vu qu'elle contient le sql_level), ça serait néfaste de retirer le code qui fonctionne actuellement.

fpeters commented

2023-01-18 18:08:28 +01:00

Ça me semble bien. La seule exception sur un tel changement serait la table sql_meta.
Vu qu'elle subit les migrations systématiquement (vu qu'elle contient le sql_level), ça serait néfaste de retirer le code qui fonctionne actuellement.

Ok mais en fait j'avais zappé la fonction de cette table sans doute parce que le code contenait déjà "IF NOT EXISTS" et,

        if not index_exists:
            cur.execute('CREATE UNIQUE INDEX IF NOT EXISTS wcs_meta_key ON wcs_meta (key)')

je me dirais que la recherche dans information_schema.tables de l'index pour avoir la variable "index_exists" pourrait être supprimée, et cette ligne systématiquement exécutée.

J'envoie le code validé ici mais si ça vaut la discussion je peux lancer ça dans un nouveau ticket.

> Ça me semble bien. La seule exception sur un tel changement serait la table sql_meta. > Vu qu'elle subit les migrations systématiquement (vu qu'elle contient le sql_level), ça serait néfaste de retirer le code qui fonctionne actuellement. Ok mais en fait j'avais zappé la fonction de cette table sans doute parce que le code contenait déjà "IF NOT EXISTS" et, ``` if not index_exists: cur.execute('CREATE UNIQUE INDEX IF NOT EXISTS wcs_meta_key ON wcs_meta (key)') ``` je me dirais que la recherche dans information_schema.tables de l'index pour avoir la variable "index_exists" pourrait être supprimée, et cette ligne systématiquement exécutée. J'envoie le code validé ici mais si ça vaut la discussion je peux lancer ça dans un nouveau ticket.

fpeters merged commit db87f1bf84 into main

2023-01-18 18:09:40 +01:00

fpeters deleted branch wip/60507-create-index-if-not-exists

2023-01-18 18:09:41 +01:00

vdeniaud referenced this pull request

2023-02-15 12:49:23 +01:00

testdef, exécuter les tests automatiquement (#74302) #110

Sign in to join this conversation.

No reviewers

No Label

No Milestone

No Assignees

2 Participants

Notifications

Due Date

The due date is invalid or out of range. Please use the format 'yyyy-mm-dd'.

No due date set.

Dependencies

No dependencies set.

Reference: entrouvert/wcs#42