Discussion :

[nox75]

Bonjour à tous,

j'ai un probleme avec un trigger qui m'embete depuis quelques jours :

J'ai une base SQL et une table avec un trigger en INSERT/UPDATE. Tout marche bien lorsque j'insere une ligne ou deux, mais lorsque je fais un import important (via DTSX), le trigger ne se déclenche qu'apres un certain nombre de lignes importées.

Avez vous connaissances d'un bug de ce genre ? Y'a-t-il un contournement/solution possible ?

Je ne sais pas si ma description du probleme est suffisante, n'hésitez pas s'il manque des indications.

[rudib]

Bonjour,

Ton trigger est-il ensembliste ? Peux-tu poster son code ?

[nox75]

Voici mon code :

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
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
34
35
36
37
38
39
40
41
42
43
44
45
46
47
48
49
50
51
52
53
54
55
56
57
58
59
60
61
62
63
64
65
66
67
68
69
70
71
72
73
74
75
76
77
78
79
80
81
82
83
84
85
86
87
88
89
90
91
92
93
94
95
96
97
98
99
100
101
102
103
104
105
106
107
108
109
110
111
112
113
114
115
116
117
118
119
120
121
122
 
set ANSI_NULLS ON
set QUOTED_IDENTIFIER ON
go
 
ALTER TRIGGER [x_lead_links_xml_imports]
   ON  [SYSADM].[X_Lead_Links]
   AFTER INSERT
AS
DECLARE 
 
@LeaNrid DECIMAL(15,0),
@cpt_nrid int,
@test VARCHAR(50),
@CompanyName VARCHAR(50),
@CompanyPC VARCHAR(50),
@ContactFN VARCHAR(50),
@ContactLN VARCHAR(50),
@ExistingCompany decimal, -- linked company/contact information
@ExistingCompanyKN decimal,
@ExistingContactFN decimal,
@ExistingContactLN decimal,
@TestExistingCompany varchar(10),
@TestExistingContact varchar(10),
@strTable varchar(3), --fctRID parameters
@societe varchar(50),
@societe_type varchar(50),
@societe_city varchar(50),
@societe_street varchar(50),
@so0_nrid decimal,
@societe_PC varchar(10),
@personne varchar(50),
@pe0_nrid decimal,
@nrid decimal
 
 
BEGIN
 
    SELECT @nrid = nrid, @LeaNrid = lea0_nrid, @CompanyName = societe_lead , @CompanyPC = societe_exist, @ContactFN = personne_lead, @ContactLN = personne_exist, @test = societe_street from inserted
 
        IF (@test = 'xml_trigger')
        BEGIN
 
                SET @ExistingCompany = null
                SET @ExistingCompanyKN = null
                SET @ExistingContactFN = null
                SET @ExistingContactLN = null
 
                SET @ExistingCompany = (select top 1 nrid from sysadm.so0 where societe = @CompanyName and code_post = @CompanyPC and template is null)
                SET @ExistingCompanyKN = (select top 1 nrid from sysadm.so0 where SOCIETE_2 = @CompanyName and code_post = @CompanyPC and template is null)
                SET @ExistingContactFN = (select top 1 nrid from sysadm.pe0 where prenom = @ContactFN and template is null)
                SET @ExistingContactLN = (select top 1 nrid from sysadm.pe0 where personne = @ContactLN and template is null)
 
        -- Insert data initialization
                SET @TestExistingCompany=(select lay.message from sysadm.set_lay lay where lay.identify = 'No' and applic_id=100000000000001)
                SET @TestExistingContact=(select lay.message from sysadm.set_lay lay where lay.identify = 'Ignore' and applic_id=100000000000001)
                SET @societe=null
                SET @so0_nrid=null
                SET @personne=null
                SET @pe0_nrid=null
                SET @societe_PC=null
 
 
                IF @ExistingCompany is not null
                BEGIN
                        SET @TestExistingCompany=(select lay.message from sysadm.set_lay lay where lay.identify = 'Yes' and applic_id=100000000000001)
                        SET @societe=@CompanyName
                        SET @so0_nrid=@ExistingCompany
                        SET @societe_type=(select type from sysadm.so0 where nrid=@ExistingCompany)
                        SET @societe_city=(select loc from sysadm.so0 where nrid=@ExistingCompany)
                        SET @societe_street=(select adresse from sysadm.so0 where nrid=@ExistingCompany)
                        SET @societe_PC=@CompanyPC
                END
                ELSE IF @ExistingCompanyKN is not null
                BEGIN
                        SET @TestExistingCompany=(select lay.message from sysadm.set_lay lay where lay.identify = 'Ignore' and applic_id=100000000000001)
                        SET @societe=@CompanyName
                        SET @so0_nrid=@ExistingCompany
                        SET @societe_type=(select type from sysadm.so0 where nrid=@ExistingCompany)
                        SET @societe_city=(select loc from sysadm.so0 where nrid=@ExistingCompany)
                        SET @societe_street=(select adresse from sysadm.so0 where nrid=@ExistingCompany)
                        SET @societe_PC=@CompanyPC
                END
 
                IF @ExistingContactFN is not null
                BEGIN
                        IF @ExistingContactLN is not null
                        BEGIN
                                SET @TestExistingContact=(select lay.message from sysadm.set_lay lay where lay.identify = 'Yes' and applic_id=100000000000001)
                                SET @personne=@ContactFN + ' ' + @ContactLN
                                SET @pe0_nrid=@ExistingContactLN
                        END
                        ELSE
                        BEGIN
                                SET @TestExistingContact=(select lay.message from sysadm.set_lay lay where lay.identify = 'Ignore' and applic_id=100000000000001)
                                SET @personne=@ContactFN + ' ' + (SELECT personne FROM sysadm.pe0 where nrid = @ExistingContactFN)
                                SET @pe0_nrid=@ExistingContactLN
                        END
                END
                ELSE IF @ExistingContactLN is not null
                BEGIN
                        SET @TestExistingContact=(select lay.message from sysadm.set_lay lay where lay.identify = 'Ignore' and applic_id=100000000000001)
                        SET @personne=(SELECT prenom FROM sysadm.pe0 where nrid = @ExistingContactFN) + ' ' + @ContactLN
                        SET @pe0_nrid=@ExistingContactLN
                END
 
                UPDATE sysadm.x_lead_links SET societe = @societe,
                                               societe_exist = @TestExistingCompany,
                                               so0_nrid = @so0_nrid,
                                               societe_type =@societe_type,
                                               societe_postalcode = @societe_PC,
                                               societe_city = @societe_city,
                                               societe_street = @societe_street,
                                               personne_exist = @TestExistingContact,
                                               personne = @personne,
                                               personne_lead = (@ContactFN + ' ' + @ContactLN),
                                               pe0_nrid = @pe0_nrid
                                               where nrid = @nrid
 
        END
 
END

[phoenix38]

Bonjour,
Ce code fonctionne au coup par coup. Pour des insertions en bloc, le select devrait être dans un curseur parcouru par le code afin de prendre en compte tous les enregistrements insérés.

[nox75]

Hm...je ne suis pas sur de comprendre.
Je ne connais pas les "insertions en bloc". Le curseur va devoir parser la table INSERTED et traiter les lignes une par une ? C'est ca ?

[phoenix38]

oui, c'est ça.

[nox75]

Tres bien je vais essayer.
Merci beaucoup pour votre aide (je mettrai en résolu des que les tests seront confirmés ^^ ) .

[nox75]

Eh ben non
Meme probleme avec cette nouvelle version :

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
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
34
35
36
37
38
39
40
41
42
43
44
45
46
47
48
49
50
51
52
53
54
55
56
57
58
59
60
61
62
63
64
65
66
67
68
69
70
71
72
73
74
75
76
77
78
79
80
81
82
83
84
85
86
87
88
89
90
91
92
93
94
95
96
97
98
99
100
101
102
103
104
105
106
107
108
109
110
111
112
113
114
115
116
117
118
119
120
121
122
123
124
125
126
127
128
129
130
131
132
133
134
135
136
137
138
139
140
141
142
143
144
145
146
147
 
set ANSI_NULLS ON
set QUOTED_IDENTIFIER ON
go
 
CREATE TRIGGER [x_lead_links_xml_imports]
   ON  [SYSADM].[X_Lead_Links]
   AFTER INSERT
AS
DECLARE 
 
@LeaNrid DECIMAL(15,0),
@cpt_nrid int,
@test VARCHAR(50),
@CompanyName VARCHAR(50),
@CompanyPC VARCHAR(50),
@ContactFN VARCHAR(50),
@ContactLN VARCHAR(50),
@ExistingCompany decimal, -- linked company/contact information
@ExistingCompanyKN decimal,
@ExistingContactFN decimal,
@ExistingContactLN decimal,
@TestExistingCompany varchar(10),
@TestExistingContact varchar(10),
@strTable varchar(3), --fctRID parameters
@societe varchar(50),
@societe_type varchar(50),
@societe_city varchar(50),
@societe_street varchar(50),
@so0_nrid decimal,
@societe_PC varchar(10),
@personne varchar(50),
@pe0_nrid decimal,
@nrid decimal
 
 
BEGIN
 
DECLARE curs_Ins CURSOR FOR
    SELECT nrid, lea0_nrid, societe_lead , societe_exist, personne_lead, personne_exist, societe_street from inserted
 
OPEN curs_Ins
FETCH curs_Ins INTO @nrid, @LeaNrid, @CompanyName, @CompanyPC, @ContactFN, @ContactLN, @test
 
WHILE @@FETCH_STATUS = 0
BEGIN 
        IF (@test = 'xml_trigger')
        BEGIN
 
                SET @ExistingCompany = null
                SET @ExistingCompanyKN = null
                SET @ExistingContactFN = null
                SET @ExistingContactLN = null
 
                SET @ExistingCompany = (select top 1 nrid from sysadm.so0 where societe = @CompanyName and code_post = @CompanyPC and template is null)
                SET @ExistingCompanyKN = (select top 1 nrid from sysadm.so0 where SOCIETE_2 = @CompanyName and code_post = @CompanyPC and template is null)
                SET @ExistingContactFN = (select top 1 nrid from sysadm.pe0 where prenom = @ContactFN and template is null)
                SET @ExistingContactLN = (select top 1 nrid from sysadm.pe0 where personne = @ContactLN and template is null)
 
        -- Insert data initialization
                SET @TestExistingCompany=(select lay.message from sysadm.set_lay lay where lay.identify = 'No' and applic_id=100000000000001)
                SET @TestExistingContact=(select lay.message from sysadm.set_lay lay where lay.identify = 'Ignore' and applic_id=100000000000001)
                SET @societe=null
                SET @so0_nrid=null
                SET @personne=null
                SET @pe0_nrid=null
                SET @societe_PC=null
 
 
                IF @ExistingCompany is not null
                BEGIN
                        SET @TestExistingCompany=(select lay.message from sysadm.set_lay lay where lay.identify = 'Yes' and applic_id=100000000000001)
                        SET @societe=@CompanyName
                        SET @so0_nrid=@ExistingCompany
                        SET @societe_type=(select type from sysadm.so0 where nrid=@ExistingCompany)
                        SET @societe_city=(select loc from sysadm.so0 where nrid=@ExistingCompany)
                        SET @societe_street=(select adresse from sysadm.so0 where nrid=@ExistingCompany)
                        SET @societe_PC=@CompanyPC
                END
                ELSE IF @ExistingCompanyKN is not null
                BEGIN
                        SET @TestExistingCompany=(select lay.message from sysadm.set_lay lay where lay.identify = 'Ignore' and applic_id=100000000000001)
                        SET @societe=@CompanyName
                        SET @so0_nrid=@ExistingCompany
                        SET @societe_type=(select type from sysadm.so0 where nrid=@ExistingCompany)
                        SET @societe_city=(select loc from sysadm.so0 where nrid=@ExistingCompany)
                        SET @societe_street=(select adresse from sysadm.so0 where nrid=@ExistingCompany)
                        SET @societe_PC=@CompanyPC
                END
 
                IF @ExistingContactFN is not null
                BEGIN
                        IF @ExistingContactLN is not null
                        BEGIN
                                SET @TestExistingContact=(select lay.message from sysadm.set_lay lay where lay.identify = 'Yes' and applic_id=100000000000001)
                                SET @personne=@ContactFN + ' ' + @ContactLN
                                SET @pe0_nrid=@ExistingContactLN
                        END
                        ELSE
                        BEGIN
                                SET @TestExistingContact=(select lay.message from sysadm.set_lay lay where lay.identify = 'Ignore' and applic_id=100000000000001)
                                SET @personne=@ContactFN + ' ' + (SELECT personne FROM sysadm.pe0 where nrid = @ExistingContactFN)
                                SET @pe0_nrid=@ExistingContactLN
                        END
                END
                ELSE IF @ExistingContactLN is not null
                BEGIN
                        SET @TestExistingContact=(select lay.message from sysadm.set_lay lay where lay.identify = 'Ignore' and applic_id=100000000000001)
                        SET @personne=(SELECT prenom FROM sysadm.pe0 where nrid = @ExistingContactFN) + ' ' + @ContactLN
                        SET @pe0_nrid=@ExistingContactLN
                END
 
        /*      print 'Lead NRID : ' + convert(char,@LeaNrid)
                print 'Company : ' + @CompanyName
                print 'Company PC : ' + @CompanyPC
                print 'Contact FN : ' + @ContactFN
                print 'Contact LN : ' + @ContactLN
                print 'Test Company : ' + @TestExistingCompany
                print 'Test Contact : ' + @TestExistingContact  */
 
 
        /*      print 'NRID : '+ convert(char,@nNRID)
                print 'RID : '+ @strRID
                print 'RMOD : '+ @strRMOD
                print '@strDMOD' + convert(char,@strDMOD)       */
 
                UPDATE sysadm.x_lead_links SET societe = @societe,
                                               societe_exist = @TestExistingCompany,
                                               so0_nrid = @so0_nrid,
                                               societe_type =@societe_type,
                                               societe_postalcode = @societe_PC,
                                               societe_city = @societe_city,
                                               societe_street = @societe_street,
                                               personne_exist = @TestExistingContact,
                                               personne = @personne,
                                               personne_lead = (@ContactFN + ' ' + @ContactLN),
                                               pe0_nrid = @pe0_nrid
                                               where nrid = @nrid
 
        END
		FETCH curs_Ins INTO @nrid, @LeaNrid, @CompanyName, @CompanyPC, @ContactFN, @ContactLN, @test
END
 
CLOSE curs_Ins
DEALLOCATE curs_Ins
 
END

Un contournement pourrait consister en un traitement des lignes non traitées lors de chaque execution du trigger, je peux repérer les lignes non traitées...mais c'est pas propre, et j'aimerais comprendre pourquoi ce trigger ne s'active qu'à mi-chemin.
Je vais continuer de creuser la question, si vous avez d'autres pistes...

[nox75]

Petite indic supplémentaire :
Le probleme n'arrive que sur un serveur, sur un autre serveur ca a l'air de passer : / (le probleme c'est que c'est sur la prod que ca passe pas)

[SQLpro]

98% de vos lignes de codes sont inutile :
1) inutile d'assigner à NULL : une variable est toujours initialisée à NULL
2) vos variables sont extraites de requêtes similaires, groupez les assignation dans un SELECT d'affectation :

Code :

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

SELECT @societe_type=type, @societe_city=loc FROM sysadm.so0 WHERE nrid=@ExistingCompany

3) réintégrez toutes vos requêtes par réemmboitage dans l'update et !
--1) il n'y a plus besoin de variables
--2) c'est naturtellement ensembliste donc pas de curseur.

A +

[nox75]

Merci, je vais essayer de réécrire le curseur en prenant en compte ces conseils.

En espérant que cela résoudra le probleme

[nox75]

Dsl je remonte encore une fois ce post.
J'ai poursuivi les investigations.
Je balance maintenant les lignes une à une dans ma table, et il s'avere que d'un seul coup, autour de la 2800eme, j'ai une colonne (personne_exist) qui passe à NULL pour toutes les lignes, meme les nouvelles insérées

Voyez vous pourquoi ca pourrait se déclencher d'un coup au milieu de nul part ?


EDIT :

Je vais tester cette solution pas tres propre d'updater les mauvaises lignes à la fin du trigger :

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
16
17
18
19
20
 
declare @Count decimal
 
select @Count=count(*) from sysadm.x_lead_links where personne_exist is null and datepart(hour,dmod) >= (datepart(hour,getdate())-2)
 
if (@Count > 0)
begin
update sysadm.x_lead_links 
set personne_exist = 
	(select 
		case 
			(select count(*) from sysadm.pe0 pe0 where pe0.personne = personne_lead and pe0.template is null) 
			when 0 then	
				(select lay.message from sysadm.set_lay lay where lay.identify = 'Ignore' and applic_id=100000000000001) 
			else 
				(select lay.message from sysadm.set_lay lay where lay.identify = 'Yes' and applic_id=100000000000001)
		end
	) 
where personne_exist is null
end

Pas tres propre mais je vois que ca.
Auriez vous d'autres suggestions ?

EDIT EDIT : en me relisant, je prends conscience que mon probleme est vraiment mal décrit.
Je vais récapituler.
Il s'agit ici d'un interfaçage entre SAP et SQL Server. Pour cette interface, on passe par du XML. L'admin SAP nous fournit les données sous forme de XML et on utilise un logiciel spécial qui genere une clé unique (le nrid) et qui insere les données dans la table indiquée (en l'occurence x_lead_links).
A chaque insertion dans x_lead_links, notre trigger fait un update (peu importe le contenu, j'ai essayé en mettant juste un update anecdotique l'effet est le meme) de la ligne insérée (on récupere le nrid, donc la clé unique, dans Inserted). C'est le seul trigger actif sur la table.
Chaque fichier XML contient une seule ligne à insérer, et les fichiers XML sont intégrés un par un.
J'en envoie environ 4000.
Jusqu'à environ 2800 lignes insérées, tout se passe bien, et d'un coup une colonne de la table passe à NULL sur tous les enregistrements insérés précédemment, et meme sur les nouveaux (alors que l'update anecdotique du trigger ne touche meme pas à la colonne en question).
Je cherche d'où peut provenir cette erreur.
J'espere que cette description plus détaillée aide à comprendre le probleme.