Discussion :

[Caidriq]

Bonjour,

Je m'excuse si une discussion existe déjà mais en cherchant depuis 2 jours je n'ai rien trouvé.
Donc normalement j'avais créée une séquence et j'avais le choix avec nextval dans la requête ou un trigger à la création de la table, on m'a demandé de ne pas passer par cette solution, question de droit.
Donc on m'a donné une fausse solution, récupérer la valeur max puis faire tout bêtement un + 1, ce que j'ai fait :

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
INSERT INTO TABLE1 (COLONNE 1, COLONNE 2, COLONNE 3, COLONNE 4)
SELECT (SELECT MAX(COLONNE1) + 1 FROM TABLE1), SYSDATE, P.COLONNE1, P.COLONNE2
FROM TABLE2 P;

Malheureusement le MAX() renvoi toujours la même valeur, j'imagine que le INSERT TO insère le total des lignes à la fin, du coup le SELECT MAX... se fait toujours sur la même ligne d’où la même valeur.
Donc existe t'il une solution ou suis-je en train de perdre mon temps ?

Merci pour tout éventuelle solution !

[al1_24]

Donc normalement j'avais créée une séquence et j'avais le choix avec nextval dans la requête ou un trigger à la création de la table, on m'a demandé de ne pas passer par cette solution, question de droit.

Que veux-tu dire par "question de droit" ?
Si tu ne peux pas créer de Séquence, tu peux peut-être créer une colonne de type IDENTITY...
Le choix de la solution avec MAX(colonne) est le moins bon des trois que tu as présentés.

[Caidriq]

Apparemment il faut des droits particuliers pour pouvoir créer des séquences et des triggers et pour la création de la table ainsi que l'ajout de ligne, le user que j'utilise ne les a pas et ils ne veulent pas en changer.
C'est la moins bonne des 3 et surtout ça ne marche pas, enfin en tout cas j'ai pas trouvé comment la faire marcher.
IDENTITY c'est quoi exactement ?

EDIT : Apparemment IDENTITY n'est accessible qu'à partir de oracle 12g.

[SQLpro]

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
6
7
8
WITH 
T0 AS (SELECT MAX(COLONNE1) + 1 AS N 
       FROM TABLE1),
T1 AS (SELECT ROW_NUMBER() OVER(ORDER BY SYSDATE) + N AS NEW_ID, SYSDATE, P.COLONNE1, P.COLONNE2
       FROM   TABLE2 P
              CROSS JOIN T)
INSERT INTO TABLE1 (COLONNE1, COLONNE2, COLONNE3, COLONNE4)
SELECT * FROM T1;

A +

[Caidriq]

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
6
7
8
WITH 
T0 AS (SELECT MAX(COLONNE1) + 1 AS N 
       FROM TABLE1),
T1 AS (SELECT ROW_NUMBER() OVER(ORDER BY SYSDATE) + N AS NEW_ID, SYSDATE, P.COLONNE1, P.COLONNE2
       FROM   TABLE2 P
              CROSS JOIN T)
INSERT INTO TABLE1 (COLONNE1, COLONNE2, COLONNE3, COLONNE4)
SELECT * FROM T1;

A +

Lorsque je l'exécute dans TOAD j'ai une exception : ORA-00928: missing SELECT keyword et il me surligne le INSERT

[skuatamad]

Il faut mettre la ligne :

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

INSERT INTO TABLE1 (COLONNE1, COLONNE2, COLONNE3, COLONNE4)

En 1er avant le WITH.

Vous êtes bien conscient que ce code ne pourra fonctionner que si vous êtes seul sur la base à faire des insertions.

[Caidriq]

Non je n'étais pas au courant, par curiosité, quelle en est la cause ? Je pense que finalement une séquence c'est vraiment le plus simple.

[al1_24]

Si un autre utilisateur/traitement ajoute une ou plusieurs lignes dans la table pendant que tu es en train d'effectuer ton import "de masse", es-tu certain que le MAX(identifiant) +n sera toujours unique ?

[SQLpro]

Si un autre utilisateur/traitement ajoute une ou plusieurs lignes dans la table pendant que tu es en train d'effectuer ton import "de masse", es-tu certain que le MAX(identifiant) +n sera toujours unique ?

Dans ce cas il suffit de monter le niveau d'isolation à SERIALIZABLE.

A +

[pachot]

Dans ce cas il suffit de monter le niveau d'isolation à SERIALIZABLE.

A +

Bonjour,
Le niveau d'isolation ne courcerne que des transactions avec plusieurs requêtes. l'exécution d'une requête fonctionne toujours en serialisable.
Ici ça ne changera rien de toute façon. Il faudrait locker la table pour utiliser ce mécanisme en multi utilisateur. La solution c'est la séquence. C'est fait pour.
Cordialement,
Franck