Use concurrency to speed up test_solver view

aledejesus / sudoku_solver

Web app that solves sudoku puzzles using classic sudoku-solving algorithms

1 stars 0 forks source link

Use concurrency to speed up test_solver view #120

Open aledejesus opened 6 years ago

aledejesus commented 6 years ago

This view calls SudokuPuzzle.solve multiple times, as more tests are added the response time will increase significantly. Use concurrency to solve all tests at the same time

aledejesus commented 6 years ago

A way to solve this would be solve puzzles in celery workers. As a provisional solution, tasks would be created and we would poll for the answer before the view returns the response. After #116 is solved, this approach would be removed and view would return as soon as celery tasks were created.

aledejesus commented 6 years ago

Reminder: enqueue tasks and continue doing what can be done before polling for the answer.

aledejesus commented 6 years ago

TODO:

finish reading celery first steps
finish reading celery Django first steps
configure celery
test celery
learn about RabbitMQ
set a backend for celery
modify test_solver to enqueue tasks and poll for the results
apply reminders
test
benchmark
create unit tests

aledejesus commented 6 years ago

TODO:

review Celery docs
learn about RabbitMQ
check why easy puzzle solving time increased. This might be related to database connections.
test
benchmark
update unit tests
create unit tests

aledejesus commented 6 years ago

TODO:

create stackoverflow question about increase in solving time
research how to test code that calls celery tasks
update unit tests
create unit tests