MaxRetries not getting set

[BillBuilt]

Job MaxRetries field is not being saved and always defaults to '23':

neoq/backends/postgres/postgres_backend.go

Line 422 in de2e101

err = tx.QueryRow(ctx, `INSERT INTO neoq_jobs(queue, fingerprint, payload, run_after, deadline)

[acaloiaro]

Thanks @BillBuilt -- confirmed. This will be fixed when #86 is merged.

[BillBuilt]

Thanks! FWIW: I could not find any references to "max retries" in any of the backends, but I wasn't searching very thoroughly in the others.

[acaloiaro]

Retries for the Redis backend are implemented in asynq. However, the mem backend also appears to not be handling max retries correctly and that'll need to be fixed as well.

I'll use this as an opportunity to increase test coverage.

[acaloiaro]

Hi @BillBuilt I'm going to go ahead and merge the postgres fix for you and open up a new issue to work on the mem backend. Thanks for the report!

[BillBuilt]

Just in time for my unit tests! Thank you! Bill

[acaloiaro]

Excellent!