Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

feat: notifica al sincronizador en tiempo real cuando hay requests pendientes por procesar #28

Merged
merged 1 commit into from
Aug 6, 2023

Conversation

AndresRamos
Copy link
Owner

En este PR se implementó SignalR en la API para poder notificar al sincronizador cuando hay solicitudes nuevas por procesar.

Cuando se crea una nueva solicitud en la API, la API notificara al sincronizador que debe consultar las solicitudes pendientes por procesar. El sincronizador estará validando cada 1 segundo estas notificaciones, pero ya no haciendo un request a la API, sino una colección de notificaciones local. El procesamiento de las solicitudes debe sentirse ahora en tiempo real.

El motivo de este cambio es que, dependiendo de la configuración de los clientes, el sincronizador estaba consultando a la API si existían solicitudes pendientes y esto estaba consumiendo varios recursos y número de peticiones por licencia.
Si el cliente configuraba el sincronizador cada 1 segundo y lo dejaba abierto las 24 horas, esto se traducía a 86,400 solicitudes por dia o 2,592,000 al mes POR CLIENTE. Esto no era sustentable.

Este cambio mejorara los tiempos de ejecución y ahorrara recursos.

…entes por procesar

De esta forma el sincronizador puede saber en tiempo real cuando hay solicitudes nuevas.
@AndresRamos AndresRamos added enhancement New feature or request web api labels Aug 6, 2023
@AndresRamos AndresRamos added this to the v2.4.0 milestone Aug 6, 2023
@AndresRamos AndresRamos self-assigned this Aug 6, 2023
@AndresRamos AndresRamos merged commit ed2a4f6 into release-v2.4.0 Aug 6, 2023
@AndresRamos AndresRamos deleted the notifica_sincronizdor_con_signalr branch August 6, 2023 08:26
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request web api
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant