SinergiaTIC / SinergiaDA

SinergiaDA is an open source analytics tool that fully integrates with SinergiaCRM
GNU Affero General Public License v3.0
4 stars 0 forks source link

Hotfix - API - Potenciar las tablas incluidas en la consulta #113

Closed jortilles closed 7 months ago

jortilles commented 7 months ago

Descripción del Cambio

Se altera el algoritmo de creación de caminos para poner primero los caminos de las tablas que salen en la consulta. De este modo, si una tabla está implicada en la consulta el camino que va por esa tabla se prioriza por delante de otros en la lista de tablas que componen los joins de la consulta.

Issue(s) resuelto(s)

Pruebas a realizar para validar el cambio

Se realizan las consultas que se indican en los issues 45 y 102 y se comprueba que el resultado es el esperado.

jortilles commented 7 months ago

Hola Juan.

Que mejoras consideras? Quieres que lo comentemos?

------------------------------------------------------------------- Juanjo Ortilles Jortilles: Open https://www.jortilles.com/Business Analytics https://www.jortilles.com/ | EDA: the most friendly analytical tool https://eda.jortilles.com/ @. @.>ti @.>lles.com @.> Tlf: +34 664 720 760

Este correo electrónico y, en su caso, cualquier fichero anexo al mismo, contiene información de carácter confidencial exclusivamente dirigida a su destinatario o destinatarios. Queda prohibida su divulgación, copia o distribución a terceros sin la previa autorización escrita del remitente, en virtud de la legislación vigente. En el caso de haber recibido este correo electrónico por error, se ruega notificar inmediatamente esta circunstancia mediante reenvío a la dirección electrónica del remitente y la destrucción del mismo.

El lun, 19 feb 2024 a las 9:13, Juan Chamizo González (< @.***>) escribió:

@.**** approved this pull request.

Todas las pruebas realizadas has sido correctas y aunque podría haber algunas mejoras no se considera que sean imprescindibles por estar en desarrollo el sistema de elección de relaciones vía árbol

— Reply to this email directly, view it on GitHub https://github.com/SinergiaTIC/SinergiaDA/pull/113#pullrequestreview-1887789995, or unsubscribe https://github.com/notifications/unsubscribe-auth/AAKHGNCR7SJQ32LMWPUM7LTYUMCSNAVCNFSM6AAAAABDN6UB5WVHI2DSMVQWIX3LMV43YUDVNRWFEZLROVSXG5CSMV3GSZLXHMYTQOBXG44DSOJZGU . You are receiving this because you authored the thread.Message ID: @.***>