Discussion :

[crocodingo]

Bonjour,

j'aimerai savoir si on peut faire plusieur action dans un forall?

actuellement je l'utilise comme ceci mais je trouve cela ridicul puisque je dois reparcourir ma table pour faire d'autre opération :

Actuellement :

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
 
PROCEDURE copy_transac_valide IS
BEGIN
   select * BULK COLLECT into transac_array from transac partition (&2) where origin <> 0;
   forall i in transac_array.first .. transac_array.last
      insert into purge_transac_tmp VALUES transac_array(i);
 
   for rec_t in C_TRANS loop
      for rec_detail in C_DETAIL(rec_t.trans_id) loop
         insert into purge_detail_tmp values (rec_detail.TRANS_ID,rec_detail.RECORD_DATE,rec_detail.NAME,rec_detail.TYPE,rec_detail.STRING_VALUE,rec_detail.NUMBER_VALUE,rec_detail.DATE_VALUE);
      end loop;
   end loop;
 
   commit;
END;

mes 2 curseurs :

Code :

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

1
2
3
4
5
6
7
8
9
 
CURSOR C_TRANS IS
	select *
	from transac partition (&2);
 
CURSOR C_DETAIL(trans NUMBER) IS
	select *
	from transac_detail partition (&2)
	where trans_id = trans;

Et je voudrais fair un truc du genre :

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
 
PROCEDURE copy_transac_valide IS
BEGIN
   select * BULK COLLECT into transac_array from transac partition (&2) where origin <> 0;
   forall i in transac_array.first .. transac_array.last
      insert into purge_transac_tmp VALUES transac_array(i);
      for rec_detail in C_DETAIL(transac_array(i).trans_id) loop
         insert into purge_detail_tmp values (rec_detail.TRANS_ID,rec_detail.RECORD_DATE,rec_detail.NAME,rec_detail.TYPE,rec_detail.STRING_VALUE,rec_detail.NUMBER_VALUE,rec_detail.DATE_VALUE);
      end loop;
 
   commit;
END;

C'est possible ou je dois laisser tomber le forall et utiliser un for classique et tout mettre dans ma boucle?

[McM]

Tu sais que tu peux faire un INSERT INTO ... SELECT .... ?
Ce sera 100 fois plus performant qu'une boucle.

[dyvim]

D'accord avec McM...

Ne jamais utiliser du code PL/SQL si tu peux le faire avec du SQL simple...

Sinon dans ton FORALL tu ne peux effectuer qu'une seule action...
Si tu veux en faire plusieurs, alors il faut utiliser un curseur "classique" (encore moins rapide en passant)

[crocodingo]

Oki merci.

En fait j'utilisait deja les curseur classique (pas tres performant effectivement) et la requete SQL directement n'est pas envisageable car les operations s'effectue sur des table de plusieurs millions d'enregistrements et sa explose pour l'insert...

Je vais rester avec mes bon vieux curseur pour l'instant.

[dyvim]

C'est quoi ton message d'"explosion" pour l'INSERT?

Parce que c'est un peu bizarre que ça marche en PL mais pas en SQL quand même...