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

Mecanismo de timeout y retries en los envios a IOTAs #72

Closed
fgalan opened this issue Nov 23, 2023 · 4 comments
Closed

Mecanismo de timeout y retries en los envios a IOTAs #72

fgalan opened this issue Nov 23, 2023 · 4 comments

Comments

@fgalan
Copy link
Contributor

fgalan commented Nov 23, 2023

Viene de #70 (comment)

Con respecto a time_sleep, también por homogeneidad haría como en el caso del CB: renombrarlo a sleep_send_batch con un valor por defecto (en el cb es 0, es decir no se espera entre envíos) y definido en el constructor para no repetirlo en las llamadas send_json y send_batch. Por otro lado, en entornos productivos es posible que se produzcan microcortes de conexión cuando se quieren enviar datos, sería aconsejable incorporar un mecanismo timeout/try similar al que se usa en el caso del CB, que tenemos un timeout (10 por defecto), post_retry_connect (3 por defecto), post_retry_backoff_factor (20 por defecto) definidos en el manager, que se pueden modificar a través del constructor y que send_batch (en este caso send_json) le da uso, en caso de tener timeouts hacer un número de retrys automáticamente, dará estabilidad en el envío de esos datos.

Este mecanismo de timeout/try queda pendiente de implementar, motivo por el cual creamos este issue.

(Los nombres de los métodos referidos arriba pudieran ser distintos a los definitivos, según evolucione la PR #70)

CC: @xavi12p @CeciliaFili

@fgalan fgalan mentioned this issue Nov 23, 2023
@xavi12p
Copy link

xavi12p commented May 16, 2024

Se hace el cambio y se pide PR: #85

@fgalan
Copy link
Contributor Author

fgalan commented May 17, 2024

Issue completada en la citada PR. Lo cerramos.

@xavi12p @CeciliaFili, ¿cerramos versión 0.12.0 con esto? ¿O vais a incorporar algún fix más antes de que vayamos a cerrar versión? Lo que nos digáis...

@fgalan fgalan closed this as completed May 17, 2024
@xavi12p
Copy link

xavi12p commented May 17, 2024

Hola, nosotros no tenemos previsto incorporar ninguna otra mejora. Se puede cerrar versión y publicar. Nos avisáis cuando esté publicada? Nos interesa esta funcionalidad para una ETL que hace envíos a plataforma vía IoTAgent. Gracias!! @fgalan

@fgalan
Copy link
Contributor Author

fgalan commented May 20, 2024

Ya estaría cerrada y publicada la nueva versión:

https://pypi.org/project/tc-etl-lib/0.12.0/

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants