[Jde-dev] bug en recordingserver

[Sara Marugan]

Finalmente creo que la mejor opci�n es dejar s�lo un id en RecordConfig, 
que se corresponda con el pid. El id de la BBDD s�lo se necesita al 
insertar un nuevo evento sobre la grabaci�n.

Ahora mismo como lo tengo es que antes de hacer la insercci�n del 
evento, consulta el id de la BBDD de la grabaci�n sobre la que quiere 
guardar el evento. Lo �nico que no me acaba de gustar es que tras la 
consulta hay que volver a conectar con la BBDD para que funcione la 
insercci�n.

Qu� os parece esta soluci�n?lo dem�s se deja tal y como estaba.



Sara Marugan wrote:
> Pens�ndolo he llegado a la conclusi�n de que se necesitan almacenar 
> ambos ids. El id de la base de datos lo necesitan tanto la aplicaci�n 
> que guarda eventos como la que los quiere consultar despu�s. Y el id que 
> viene del pid lo necesita recorder para matar el proceso de grabaci�n.
>
> Ahora mismo como est� (sin el cambio) las aplicaciones s�lo pueden 
> acceder al id del pid, que es lo que se almacena.
>
> As� que todo apunta a que hay que introducir el campo recoding_id en la 
> configuracion de una grabaci�n.
>
>
>
> Roberto Calvo wrote:
>   
>> 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
>>>     
>>>       
>>   
>>     
>
> _______________________________________________
> Jde-developers mailing list
> Jde-developers en gsyc.es
> http://gsyc.escet.urjc.es/cgi-bin/mailman/listinfo/jde-developers
>