Discussion :

[tartine]

Bonjour

Voici une fonction que j'ai creer :

http://pastebin.com/6h95N9Sf

c'est un trigger qui intervient Before Insert/update
Le trigger fonctionne ainsi que la fonction et l'insertion.
Le probleme survient quand je fais une requête de ce type :

Code :

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

1
2
 
select count(*),country from http group by country;

Le resultat est incorect ( du style : UK=12 FR=3 UK=2 UK=4 ) en gros : il ne regroupe pas (ne trie pas).

Voila, j'ai aussi testé avec du INT et ça marche tres bien ( comptage et regroupage).

Si quelqu'un a une idée

Tartine

[estofilo]

Quel est le rapport entre le trigger et le group by qui ne renvoie pas les résultats attendus?

[tartine]

Le trigger est sur la table sur laquelle j'effectue mes requêtes.
et c'est une valeur que je switch dans le trigger qui me donne les résultats inattendus.
c'est pourquoi je pense que j'ai mal fais ma fonction.
voila.

[estofilo]

C'est une fonction assez compliquée mais déjà les 2 dernières lignes paraissent bizarres

Code :

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

1
2
3
 
       rettuple=SPI_modifytuple(rel,newtuple,1,&ret,(Datum *)&serial,NULL);
       return PointerGetDatum(newtuple);

en ce sens que ça met la ligne de résultat dans rettuple mais la valeur n'est pas utilisée, au lieu de ça c'est newtuple qui est renvoyé.

Quoiqu'il en soit, le group by qui ne fonctionne pas c'est extrêmement étonnant comme résultat, à moins qu'il n'y ait une explication du style caractères invisibles à l'intérieur des champs, ça voudrait dire que les données sont complètement vrillées.
Pour vérifier si le contenu affiché est propre, on peut utiliser la fonction octet_length() sur le champ à problème:

Code :

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

1
2
 
SELECT country,octet_length(country),count(*) FROM http GROUP BY country, octet_length(country);

Il serait intéressant de voir s'il y a des doublons dans les 2 colonnes de gauche.
S'il y a un index sur country, il peut être intéressant de réindexer aussi des fois qu'il soit corrompu.

[tartine]

Oui, c'est vrai, la dernière ligne ne marche pas, c'est en fait pour les tests : la ligne est changée avant et j'utilise le premier changement : on règle un problème a la fois .

En ce qui concerne la longueur des strings

Code :

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

1
2
 
SELECT country,octet_length(country),count(*) FROM http GROUP BY country, octet_length(country);

Cela me retourne des valeurs plutot enormes

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
 United States |           46 |     1
 United States |           46 |     1
 nited States  |           41 |     1
 nited States  |           41 |     1
 nited States  |           41 |     1
 nited States  |           41 |     1
 United States |           46 |    12
 United States |           46 |     2
 United States |           46 |     1
 nited States  |           41 |     2
 nited States  |           41 |     2
 nited States  |           41 |     1
 nited States  |           41 |     1
 United States |           46 |     2
 United States |           46 |     1

comparé a une colonne qui fonctionne :

Code :

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

1
2
3
4
5
6
7
8
9
10
11
 
SELECT hostname,octet_length(hostname),count(*) FROM http GROUP BY hostname, octet_length(hostname);
       hostname        | octet_length | count 
-----------------------+--------------+-------
 t3.gstatic.com        |           14 |     2
 cdn0.knowyourmeme.com |           21 |     2
 cdn1.knowyourmeme.com |           21 |     4
 cdn2.knowyourmeme.com |           21 |     4
 a0.rbcdn.com          |           12 |     2
 www.google.ch         |           13 |    12
 knowyourmeme.com      |           16 |     4

Comment régler ce problème ? est-ce que le palloc de mon trigger en C fait des misères ?
la strlen de mon trigger m'indique une longueur de 13. pourquoi postgres me le fait différemment ?

[estofilo]

Je ne suis pas familier de l'API en C de postgres, mais j'ai l'impression qu'à l'appel à SPI_modifytuple(), le fait de faire simplement un cast en Datum* de ton pointeur sur chaine avec la nouvelle valeur, ce n'est pas ce qu'il faudrait faire.
Par exemple dans ce bout de code:

Code :

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

1
2
3
4
 
ret=SPI_fnumber(tupdesc, "country");
ereport( INFO,( errcode( ERRCODE_FEATURE_NOT_SUPPORTED ),errmsg( serial_num )));
newtuple=SPI_modifytuple(rel,rettuple,1,&ret,(Datum *)&serial_num,NULL);

Il faudrait plutôt faire qqch du genre

Code :

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

1
2
3
4
5
 
Datum new_data;
...
new_data = PointerGetDatum(textin(serial_num));
newtuple=SPI_modifytuple(rel,rettuple,1,&ret, &new_data, NULL);

[tartine]

Parfait ! j'ai résolu mon problème : voici la portion du code qui posait problème ( merci à estofilo pour le tuyau )

Code :

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

1
2
 
newtuple=SPI_modifytuple(rel,rettuple,1,&ret,(Datum *)&serial_num,NULL);

a transformer en :

Code :

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

1
2
3
4
5
6
 
	DatumBuffer = palloc(sizeof(Datum));
 
	DatumBuffer = DirectFunctionCall1(textin,CStringGetDatum(serial_num));
 
	newtuple=SPI_modifytuple(rel,rettuple,1,&ret,&DatumBuffer,NULL);

cependant, il y a beaucoup de warning a la compilation, mais, le tout marche donc : résolue.
Merci a tous!

Tartine