Skip to content

Conversation

@polsala
Copy link
Member

@polsala polsala commented Jan 31, 2024

Mejorar el sistema de connexiones para evitar problemas en redes con connexiones inestables o servidores redis relativamente saturados.

  • Se maniente la connexión activa
  • Se reintenta la connexión tras un timeout
  • Se espera hasta 10 segundos para esablecer connexión

@polsala polsala self-assigned this Jan 31, 2024
@polsala polsala marked this pull request as draft January 31, 2024 19:08
@polsala polsala force-pushed the improve_connections branch from 025cb31 to 5f6eb77 Compare January 31, 2024 19:11
@polsala polsala requested review from a team, ecarreras and tinogis January 31, 2024 19:33
@polsala
Copy link
Member Author

polsala commented Jan 31, 2024

Working on:

  • Binefar

# socket_timeout=5, # Timeout for socket operations, in seconds
socket_keepalive=True, # Keeps the connection alive
retry_on_timeout=True # Retries if a timeout is encountered
)
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Es podria definir els paràmetres per defete tipus:

params = {
    'sockent_connect_timeout': 10,
    'socket_keepalive': True,
    'retry_on_timeout': True,
}

i després quan fem la connexió passar-ho tipus:

redis_conn = from_url(config['redis_url'], **params)
...
redis_conn = Redis(**params)

D'aquesta forma si volem canviar aquests paràmetres per defecte només tenim un lloc i potser queda una mica més clar?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants