Mettre à jour un Champ

[AbouZaid]

salut à tous,

Je reviens vers vous car j'ai du mal avec une requête!!

J'ai une table qui a plusieurs colonnes dont (NOM_PIECE,FLAG_LIGNE et FLAG_PIECE(initialement à '0') ) et je voudrais mettre à jour FLAG_PIECE dans toute la table comme suit :

En prenant Pièce par pièce : FLAG_PIECE devra être = à MAX(FLAG_LIGNE) après MAJ , mais si toutes les lignes d'une même pièce ont FLAG_LIGNE = '0', ce n'est pas la peine de faire la MAJ puisque FLAG_PIECE est initialement = '0'.

J'ai essayé de le faire avec un curseur :

Code :

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
curseur IS 
SELECT count(*),NOM_PIECE
FROM TABLE
GROUP BY NOM_PIECE;
 
FOR w_cur IN curseur loop
 
SELECT MAX(FLAG_LIGNE) INTO var
FROM TABLE
WHERE NOM_PIECE = cur.NOM_PIECE;
 
IF var <> '0' then
UPDATE TABLE
SET FLAG_PIECE = var
WHERE STATUS = 'NEW'
AND NOM_PIECE = cur.NOM_PIECE;
end IF;
 
end loop;

Mais ces instructions tournent depuis une heure.

La table fait plus de 5 millions de lignes. et bien sur il y a deux index :

le premier sur (STATUS, NOM_PIECE)
le second sur (NOM_PIECE).

[skuatamad]

Quelle version d'oracle ? Si 10g+ utilise merge :

Code :

1
2
3
4
5
6
7
8
9
merge INTO TABLE t
USING (SELECT NOM_PIECE, MAX(FLAG_LIGNE) AS max_flag
         FROM TABLE
        WHERE STATUS = 'NEW'
        GROUP BY NOM_PIECE
       HAVING MAX(FLAG_LIGNE) <> '0') u
   ON (t.NOM_PIECE = u.NOM_PIECE)
 when matched then UPDATE
  SET t.FLAG_PIECE = u.max_flag

Sinon il faut utiliser UPDATE mais ce sera moins performant, quelque chose comme ça mais je ne suis pas sûr du HAVING de la clause EXISTS :

Code :

1
2
3
4
5
6
7
UPDATE TABLE t
   SET t.FLAG_PIECE = (SELECT MAX(u.FLAG_LIGNE) FROM TABLE u WHERE u.NOM_PIECE = t.NOM_PIECE AND STATUS = 'NEW')
 WHERE EXISTS (SELECT 1 FROM TABLE u
                WHERE u.NOM_PIECE = t.NOM_PIECE
                  AND STATUS = 'NEW'
                GROUP BY u.NOM_PIECE
               HAVING MAX(u.FLAG_LIGNE) <> '0')

En tout cas, ce qui est sûr, c'est qu'il ne faut pas utiliser de curseur, une seule requête suffit.

[edit]j'avais oublié le when matched then UPDATE...

[Waldar]

Évidemment que c'est lent, vous avez un premier curseur pour récupérer vos noms de pièce, puis une requête appelée à chaque ligne pour récupérer le max, et enfin la mise à jour qui tombe.

SQL est un langage ensembliste.
Il faut que vous le compreniez sinon vous n'aurez jamais de bonnes performances.

Tout votre bloc s'écrit ainsi :

Code :

1
2
3
4
5
6
7
8
9
merge INTO matable mt1
USING (  SELECT nom_piece, max(flag_ligne) AS flag_piece
           FROM matable
       GROUP BY nom_piece
         HAVING max(flag_ligne) <> '0') mt2
   ON (mt1.nom_piece = mt2.nom_piece)
 when matched then UPDATE
  SET mt1.flag_piece = mt2.flag_piece
WHERE mt1.STATUS = 'NEW';

[skuatamad]

Au fait Waldar a raison, le WHERE mt1.STATUS = 'NEW' doit être placé après le SET pour correspondre à ton process initial et pas dans la requête USING

[AbouZaid]

Merciiiiiiii, je vais l'essayer de suite!!!

[AbouZaid]

j'ai essayé de comprendre l'instruction Merge , mais je n'y arrive pas.

Qlq aurait la gentillesse de me donner un exemple bien clair?

merci