Skip to content

Corrige l’extraction du type SQL pour la conversion en Table Schema

Ronan Amicel requested to merge fix-sql-type-not-null into main

Problème

En ajoutant le support de types en plusieurs mots (comme DOUBLE PRECISION et CHARACTER VARYING) dans !477 (merged) j’avais cassé le cas où il y avait des contraintes à la suite du type lui-même (comme NOT NULL).

Solution

On rend la regex plus restrictive. Au passage on passe le flag VERBOSE pour pouvoir la mettre sur plusieurs lignes.

Autres approches envisagées

J’ai eu envie d’utiliser sqlglot pour parser le type plutôt que de réinventer la roue, mais je n’ai pas trouvé de point d’entrée pour faire ça simplement.

Edited by Ronan Amicel

Merge request reports