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

282 validar borken url paralelizando #287

Merged
merged 16 commits into from Oct 8, 2019

Conversation

AWolfsdorf
Copy link
Contributor

Se agrego una clase threading_helper la que enviandole una funcion, cantindad de threads y una lista de datos, hace un mapeo de la lista con sus correspondientes salidas si el dato i-esimo se usara en la funcion.

Este procedimiento no funciona con los comportamientos de Cassette ya que el mismo consume recursos fisicos y no es compatible con la concurrencia, para eso, en los tests que usan esta funcion se monkeypatchea la constante CANT_THREADS_BROKEN_URL_VALIDATOR en 1 para que no genere errores.

closes #282

@AWolfsdorf AWolfsdorf self-assigned this Oct 7, 2019
@coveralls
Copy link

coveralls commented Oct 7, 2019

Coverage Status

Coverage increased (+0.1%) to 83.817% when pulling 224e657 on 282-validar-borken-url-paralelizando into 879da46 on master.

@lucaslavandeira
Copy link
Contributor

Revisar tiempos de ejecución de los tests, algún tema de threading demora la salida del programa por unos 40-60 segundos

@AWolfsdorf AWolfsdorf merged commit a4a7034 into master Oct 8, 2019
@AWolfsdorf AWolfsdorf deleted the 282-validar-borken-url-paralelizando branch October 8, 2019 18:51
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

Successfully merging this pull request may close these issues.

Implementar chequeo de broken URLs paralelizando los chequeos
5 participants