fix: solo un trabajador por fallo #2210

Merged
jgallego merged 4 commits from 6782-productionError_add into dev 2024-03-27 06:24:26 +00:00
Owner
No description provided.
jgallego added 1 commit 2024-03-26 07:53:23 +00:00
gitea/salix/pipeline/pr-dev There was a failure building this commit Details
e47b8c1056
fix: solo un trabajador por fallo
jgallego requested review from carlosap 2024-03-26 07:53:28 +00:00
jgallego added 1 commit 2024-03-26 08:47:10 +00:00
gitea/salix/pipeline/pr-dev This commit looks good Details
b978833a64
fix: #6782 with con esquema
carlosap requested changes 2024-03-26 09:06:03 +00:00
Dismissed
@ -47,0 +37,4 @@
WITH rankedWorkers AS (
SELECT sm.id,
st.workerFk,
ROW_NUMBER() OVER(PARTITION BY sm.id ORDER BY s2.`order`) AS rnk
Member

en este caso RANK() seria más apropiado?

en este caso RANK() seria más apropiado?
Author
Owner

ChatGpt, pero crec que el rank en el meu cas no em val.

Sí, puedes utilizar la función RANK() en lugar de ROW_NUMBER() para lograr un propósito similar, pero hay una diferencia importante entre estas dos funciones que puede afectar el resultado dependiendo de lo que necesitas:

ROW_NUMBER() asigna un número único a cada fila dentro de la partición de acuerdo con el criterio de ordenación especificado. Esto significa que incluso si dos filas tienen el mismo valor en las columnas de ordenación, cada una recibirá un número de fila diferente.

RANK() asigna el mismo rango a filas que tienen los mismos valores en las columnas de ordenación, con huecos en la secuencia de rangos si hay empates. Por ejemplo, si dos filas están empatadas en el primer lugar, ambas recibirán el rango 1, y el siguiente rango sería 3, no 2.

Dependiendo de tu caso de uso específico, el uso de RANK() podría significar que si hay varios trabajadores con el mismo valor de orden para un id de saleMistake, todos recibirán el mismo rango. Si deseas considerar a todos los trabajadores que comparten el valor más alto (o más bajo, según el orden) en el conjunto de order para un id específico, RANK() sería la función adecuada.

ChatGpt, pero crec que el rank en el meu cas no em val. Sí, puedes utilizar la función RANK() en lugar de ROW_NUMBER() para lograr un propósito similar, pero hay una diferencia importante entre estas dos funciones que puede afectar el resultado dependiendo de lo que necesitas: ROW_NUMBER() asigna un número único a cada fila dentro de la partición de acuerdo con el criterio de ordenación especificado. Esto significa que incluso si dos filas tienen el mismo valor en las columnas de ordenación, cada una recibirá un número de fila diferente. RANK() asigna el mismo rango a filas que tienen los mismos valores en las columnas de ordenación, con huecos en la secuencia de rangos si hay empates. Por ejemplo, si dos filas están empatadas en el primer lugar, ambas recibirán el rango 1, y el siguiente rango sería 3, no 2. Dependiendo de tu caso de uso específico, el uso de RANK() podría significar que si hay varios trabajadores con el mismo valor de orden para un id de saleMistake, todos recibirán el mismo rango. Si deseas considerar a todos los trabajadores que comparten el valor más alto (o más bajo, según el orden) en el conjunto de order para un id específico, RANK() sería la función adecuada.
jgallego marked this conversation as resolved
@ -47,0 +39,4 @@
st.workerFk,
ROW_NUMBER() OVER(PARTITION BY sm.id ORDER BY s2.`order`) AS rnk
FROM vn.saleMistake sm
JOIN vn.saleTracking st ON sm.saleFk = st.saleFk
Member

esquema vn se puede eliminar

esquema vn se puede eliminar
Author
Owner

dins dels WITH si no es posa l'esquema myt falla.

dins dels WITH si no es posa l'esquema myt falla.
jgallego marked this conversation as resolved
@ -47,0 +46,4 @@
WHERE t.shipped BETWEEN vDatedFrom AND vDatedTo
AND s2.code IN ('OK', 'PREVIOUS_PREPARATION', 'PREPARED', 'CHECKED')
)
SELECT workerFk, COUNT(*) AS errors
Member

El AS no es necesario

El AS no es necesario
jgallego marked this conversation as resolved
jgallego added 1 commit 2024-03-26 13:03:02 +00:00
gitea/salix/pipeline/pr-dev This commit looks good Details
1472fc6fc2
feat: #6782 sin as
jgallego requested review from carlosap 2024-03-26 13:03:27 +00:00
carlosap approved these changes 2024-03-26 17:54:39 +00:00
jgallego added 1 commit 2024-03-27 06:22:51 +00:00
gitea/salix/pipeline/pr-dev This commit looks good Details
c3aa4c9355
Merge branch 'dev' into 6782-productionError_add
jgallego merged commit 075259dee9 into dev 2024-03-27 06:24:26 +00:00
jgallego deleted branch 6782-productionError_add 2024-03-27 06:24:26 +00:00
Sign in to join this conversation.
No reviewers
No Milestone
No Assignees
2 Participants
Notifications
Due Date
The due date is invalid or out of range. Please use the format 'yyyy-mm-dd'.

No due date set.

Dependencies

No dependencies set.

Reference: verdnatura/salix#2210
No description provided.