Optimizacion De Rendimiento

[[fjcg02]]

Hola,
sería interesante que pusieras todo el SP o la vista. Además, la estructura de las tablas y los índices.

He estado revisando, y ahora no se si utilizas la condición "Like :CALLED"

Si siempre miras por :CALLED+'XXX' o CALLED+'%', te convendría añadir un campo con esa información para evitar utilizar like, que es terriblemente ineficiente. Si no puedes, sería mejor utilizar left(PLAT.DESTINO,LONGITUDCALCULADA) en la condición que el like.

Ya nos contarás.

Saludos

[juanlito]

De primera voy a poner la vista, este codigo solo como vista me funciona bien si meto un valor que pertenece a la tabla lo muestra una sola vez y si meto uno que no pues no lo muestra.

Código SQL [-]
CREATE OR ALTER VIEW VISUALIZA(
    CALLINGNUMBER)
AS
SELECT FIRST 1 (CALLINGNUMBER) FROM GUAN WHERE ((CALLINGNUMBER) = '956303030')
;

Ahora os pongo el codigo del SP

Código SQL [-]
 FOR SELECT PLAT.linea,PLAT.destino, PLAT.fecha_hora_inicio FROM PLAT
 WHERE (PLAT.FECHA_HORA_INICIO > :F_INICIO) AND (PLAT.FECHA_HORA_INICIO < :F_FIN)
 INTO :LINEA, ESTINO, :HORA_LLAMADA    DO
 BEGIN
  SELECT FIRST 1 (GUAN.CALLINGNUMBER) FROM GUAN WHERE ((GUAN.CALLINGNUMBER) = :LINEA)
  INTO :VECE;
   if (:VECE IS NOT NULL ) then
/*   SELECT count (GUAN.CALLINGNUMBER) FROM GUAN WHERE (GUAN.CALLINGNUMBER = :LINEA)
  INTO :VECES;
   if (:VECES <> 0 ) then*/
   BEGIN
       lon =  CHAR_LENGTH (ESTINO);
       DEST = ESTINO;
       if (:lon > 9) then
       begin
         DEST = left(EST,:lon-4);
       end
       DEST = EST ||'%';
       SELECT COUNT (GUAN.CALLINGNUMBER) FROM GUAN WHERE (((GUAN.CALLINGNUMBER) = :LINEA) AND ((GUAN.CALLEDNUMBER) like EST) AND ((GUAN.CALLTIME) >DateAdd(hour,-2,:HORA_LLAMADA) And (GUAN.CALLTIME)hour,2,:HORA_LLAMADA)))
       INTO :VECES ;
       if (:VECES = 0 ) then
       begin
          suspend;
       end
 END
 END

Los parametros de entrada son F_INICIO y F_FIN que son de tipo TIMESTAMP.

Los parametros de salida son LINEA,DESTINO y HORA_LLAMADA, que son de tipo VARCHAR(10), VARCHAR(20) y TIMESTAMP respectivamente.

Las variables son VECES, LON con tipo INTEGER, y VECE y DEST que son VARCHAR(40) y VARCHAR(20).

Usando el select que aparece entre /* */ el SP me funciona correctamente pero he comprobado que en ese select recorro toda la tabla cuando en realidad necesito solo que me compruebe si el parametro esta, con lo que cuando lo encontrase por primera vez deberia de parar, porque la tabla ira creciendo con miles de filas.


Aqui os pongo la estructura de las tablas, los campos VARCHAR(255) se crearon asi porque la tabla se creo automaticamente asi al importar los datos, no creo que eso me genere el problema porque realmente los datos que gestiono entran en los
VARCHAR(10)..VARCHAR(20).. que yo uso

Código SQL [-]
 
CREATE TABLE GUAN (
    COLSORT              VARCHAR(255) CHARACTER SET UNICODE_FSS,
    PHONEID              VARCHAR(255) CHARACTER SET UNICODE_FSS,
    CALLINGNUMBER        VARCHAR(255) CHARACTER SET UNICODE_FSS,
    CALLEDNUMBER         VARCHAR(255) CHARACTER SET UNICODE_FSS,
    CALLTIME             TIMESTAMP,
    CALLDURATION         VARCHAR(255) CHARACTER SET UNICODE_FSS,
);
 

/******************************************************************************/
/*                                  Indices                                   */
/******************************************************************************/
CREATE INDEX GUAN_CALLINGNUMBER_NEW ON GUAN (CALLINGNUMBER);
CREATE INDEX GUAN_CALLTIME_NEW ON GUAN (CALLTIME);
CREATE INDEX GUAN_PHONEID_NEW ON GUAN (PHONEID);

y la de la otra tabla:

Código SQL [-]

CREATE TABLE PLAT (
    ID_LLAMADA           INTEGER NOT NULL,
    LINEA                VARCHAR(255) CHARACTER SET UNICODE_FSS,
    FECHA_HORA_INICIO    TIMESTAMP,
    DESTINO              VARCHAR(255) CHARACTER SET UNICODE_FSS,
    ID_DESTINO           VARCHAR(255) CHARACTER SET UNICODE_FSS,
    DURACION             INTEGER,
);
 

/******************************************************************************/
/*                                Primary Keys                                */
/******************************************************************************/
ALTER TABLE PLAT ADD PRIMARY KEY (ID_LLAMADA);

/******************************************************************************/
/*                                  Indices                                   */
/******************************************************************************/
CREATE INDEX PLAT_LINEA ON PLAT (LINEA);
CREATE INDEX PLAT_TIEMPO ON PLAT (FECHA_HORA_INICIO);

Espero que con esta informacion podamos ver donde esta el problema, porque no entiendo porque en el VIEW me funciona el select y en el SP no.

Un saludo

Gracias

[juanlito]

De primera voy a poner la vista, este codigo solo como vista me funciona bien si meto un valor que pertenece a la tabla lo muestra una sola vez y si meto uno que no pues no lo muestra.


Código SQL [-]
CREATE OR ALTER VIEW VISUALIZA(
CALLINGNUMBER)
AS
SELECT FIRST 1 (CALLINGNUMBER) FROM GUAN WHERE ((CALLINGNUMBER) = '956303030')
;




Ahora os pongo el codigo del SP


<DIV class=frame_codigo_sql>Código SQL [-]<DIV id=sql_div_4ef2f4c3b3b6d class=texto_codigo_sql>
FOR SELECT PLAT.linea,PLAT.destino, PLAT.fecha_hora_inicio FROM PLAT
WHERE (PLAT.FECHA_HORA_INICIO > :F_INICIO) AND (PLAT.FECHA_HORA_INICIO < :F_FIN)
INTO :LINEA, ESTINO, :HORA_LLAMADA DO
BEGIN
SELECT FIRST 1 (GUAN.CALLINGNUMBER) FROM GUAN WHERE ((GUAN.CALLINGNUMBER) = :LINEA)
INTO :VECE;
if (:VECE IS NOT NULL ) then
/* SELECT count (GUAN.CALLINGNUMBER) FROM GUAN WHERE (GUAN.CALLINGNUMBER = :LINEA)
INTO :VECES;
if (:VECES <> 0 ) then*/
BEGIN
lon = CHAR_LENGTH (ESTINO);
DEST = ESTINO;
if (:lon > 9) then
begin
DEST = left(EST,:lon-4);
end
DEST = EST ||'%';

SELECT COUNT (GUAN.CALLINGNUMBER) FROM GUAN WHERE (((GUAN.CALLINGNUMBER) = :LINEA) AND ((GUAN.CALLEDNUMBER) like EST) AND ((GUAN.CALLTIME) >DateAdd(hour,-2,:HORA_LLAMADA) And (GUAN.CALLTIME)

[juanlito]

Se duplico la respuesta, lo siento, no se como puedo eliminar una de ellas.

Un saludo y lo siento.

[Casimiro Notevi]

No olvides poner etiquetas al código, ejemplo:



.

[[fjcg02]]

Prueba con esto

Código SQL [-]
 SELECT DISTINCT PLAT.linea,PLAT.destino, PLAT.fecha_hora_inicio FROM PLAT
 INNER JOIN GUAN ON (PLAT.LINEA=GUAN.CALLINGNUMBER)
 WHERE 
 (PLAT.FECHA_HORA_INICIO BETWEN :F_INICIO AND :F_FIN) AND
 ( GUAN.CALLTIME BETWEN DATEADD(HOUR,-2,:F_INICIO) AND DATEADD(HOUR,2,:F_FIN)

Te quitas las dos primeras select y una parte de la tercera. Es decir, cruzamos las tablas PLAT y GUAN, con destino iguales, con fecha_hora_inicio entre el rango y con CALLTIME dos horas antes o despues.

Pongo DISTINCT para que sólo me saque un registro por combinación de los tres campos, obviando los duplicados.

Esto tiene que ir rápido, tienes indices en los campos por los que cruzamos.


Ahora nos queda el tema de DESTINO, que si es de longitud mayor que 9 te quedas con el campo menos los 4 ultimos caracteres, y si no, con todos.

Tendrías que probar algo similar a esto

Código SQL [-]
 
 AND (CASE 
         WHEN LENGTH(PLAT.DESTINO) > 9
        THEN
           CASE
             WHEN 
                LEFT(PLAT.DESTINO,LENGTH(PLAT.DESTINO)-4)=LEFT(CALLEDNUMBER,LENGTH(PLAT_DESTINO)-4) THEN
               1
             ELSE
                0
            ENDCASE
        ELSE
           CASE
             WHEN 
               LEFT(PLAT.DESTINO,LENGTH(PLAT.DESTINO))=LEFT(CALLEDNUMBER,LENGTH(PLAT_DESTINO)) THEN
              1
            ELSE
              0
            ENDCASE
      ENDCASE ) = 1

en las condiciones. No se si tragará, pero es cuestion de probarlo. Si no funciona sería cuestión de ponerlo en un campo calculado, y en la where preguntar por ese campo.

La ultima parte es un poco liosa, pero creo que puede entenderse. No estoy seguro de lo que quieres filtrar, si pones un juego de ensayo, podríamos ayudarte a cambiar la consulta para darle otea orientación.

Un saludo