source-academy / backend / 32fddf600659d3c6dd03f58dfbdbc742c3313331-PR-1363

87%

master: 87%

Build Type

Pull #1363

github

Committed by RichDom2185

Commit Message

Give autograder result storage its own queue

`ResultStoreWorker` was `use Que.Worker, concurrency: 5` before the move
to Oban, deliberately separate from the grading work so that database
writes stayed bounded -- as its `@moduledoc` still says:

    Separate worker is created with lower concurrency on the assumption
    that autograding time >> db IO time so as to reduce db load.

Putting it on the same `autograder: 20` queue as `LambdaWorker` lost
both halves of that: result jobs now queue behind a batch's Lambda
invocations, and up to 20 of them can then write to the database at
once.

Restore the separation with a dedicated `autograder_results: 5` queue.

Reported in review: PR #1363.

Pull Request Pull Request #1363: Upgrade most dependencies and modernize codebase

Coverage Stats

82 of 103 new or added lines in 17 files covered. (79.61%)

5 existing lines in 4 files now uncovered.

3997 of 4569 relevant lines covered (87.48%)

6891.55 hits per line