[Jde-dev] bug en recordingserver

[Roberto Calvo]

Sobre los eventos aún no he trabajado con la ultima versión. Pero te
comento unas cosas sobre los ids:

 * El id_rec es el pid del proceso que realiza la grabación, sólo tiene
sentido usarlo cuando la grabación está en curso (para detener el
proceso).
 * El id es el identificador único de grabación, que se utiliza para
todo lo demás en grabaciones.

El código que me pegas, es justo el primer caso, se está lanzando la
grabación, recoge el pid de proceso. En ese mismo momento el recorder no
sabe nada de ID's de base de datos, eso lo sabe el recordingManager.

Según lo que comentas, necesitamos tener en el RecorderConfig dos
atributos, uno que sea el id normal, y otro el id_rec ¿Con eso se
solucionaría el problema, verdad?

Además, fíjate que cuando haces un getRecordings, el id si introduce el
correcto, el id único de la BBDD. Si te parece cuéntame un poco más el
problema que tienes y el contexto y vemos como solucionarlo, aunque lo
que he puesto en el párrafo anterior tiene sentido que lo hagamos.

Mirando el código, creo que el problema pueda estar en que utilizas el
id que devuelve el RecorderConfig al empezar una grabación, para
establecer los eventos de esa grabación ¿es así? Si es así, creo que la
solución pasa por tener un solo campo id en el RecorderConfig, y que sea
el propio recordingManager quien se capaz de obtener el PID, que desde
luego tiene mucho más sentido.

Confírmame esto último que te comento, y lo cambiamos. 

Gracias por el reporte!
un saludo!

El jue, 22-04-2010 a las 13:21 +0200, Sara Marugan escribió:
> Hola,
> 
> haciendo pruebas con recordingserver me daba error al intentar guardar 
> un evento sobre una grabación previamente guardada. Concretamente es que 
> no coincidía el campo recording_id del evento con el id de la grabación. 
> Las grabaciones tienen un id y un recording_id. La foreing key de los 
> eventos necesita apuntar al id de una grabación, no al recording_id, que 
> es como está ahora.
> 
> Pongo aquí el snip de código de recordingserver.cpp (línea 78):
> 
>           // Read the PID
>           read (descPipe[0], &pid_rec, sizeof(int));
>           //printf("#### rid from insert %d\n",rid);
>           recConfig->id = pid_rec;
> 
>           // Log recording
>           int rid=recLog->startRecording(recConfig);
> 
>           //return pid_rec;
>           return rid;
> 
> Cambiando esto funciona bien. Creo el ticket y subo el parche?
> 
> Un saludo.
> 
> Sara.
> _______________________________________________
> Jde-developers mailing list
> Jde-developers en gsyc.es
> http://gsyc.escet.urjc.es/cgi-bin/mailman/listinfo/jde-developers

-- 
Roberto Calvo Palomino          | Libre Software Engineering Lab (GSyC)
Tel: (+34) 91 488 85 23         | Universidad Rey Juan Carlos
rocapal en libresoft.es            | Edif. Departamental II - Despacho 116
http://libresoft.es/            | c/Tulipán s/n 28933 Móstoles (Madrid)

GPG-KEY: http://gsyc.es/~rocapal/rocapal.gpg
------------ próxima parte ------------
Se ha borrado un mensaje que no está en formato texto plano...
Nombre     : no disponible
Tipo       : application/pgp-signature
Tamaño     : 198 bytes
Descripción: Esta parte del mensaje =?ISO-8859-1?Q?est=E1?= firmada
 digitalmente
Url        : http://gsyc.escet.urjc.es/pipermail/jde-developers/attachments/20100422/9cc12fe3/attachment.pgp