Ian Lewis

exception() shouldn't mess with the source argument if passed.

2

If the source argument is passed to the exception() method in LoggingWrapper, it should be used as is rather than inserting the request path or "Exception".

Worker # and worker concurrency is ambiguous

1

Currently there are multiple ways to set a limit on the number of concurrently of running tasks. 1. Run more workers asynchronously via `LaunchAsync` in which each worker will call...

Semaphore channels don't need to be initialized

2

Semaphore channels can be push style rather than pull. For example: https://github.com/RichardKnop/machinery/blob/7ccba3d50e4edd1489bd0ab1743534920deb093b/v2/brokers/redis/goredis.go#L97-L102 Can be implemented like: ```golang pool := make(chan struct{}, concurrency) select{ case pool

Fixes #737

Reads on maps in the eager backend are not protected from concurrent read & writes. Ex: https://github.com/RichardKnop/machinery/blob/master/v2/backends/eager/eager.go#L160

Results aren't expired from the eager backend so it can cause a memory leak if used for anything other than limited use.

Concurrency set on a worker passes concurrency to the broker implementation in `StartConsuming`. Each broker implementation returns an error to the worker if any concurrent task fails in which the...

The eager broker's implementation of `StartConsuming` simply returns retry: true with a nil error. https://github.com/RichardKnop/machinery/blob/master/v2/brokers/eager/eager.go#L31-L34 ```go // StartConsuming enters a loop and waits for incoming messages func (eagerBroker *Broker) StartConsuming(consumerTag...

MacOS support

4

gVisor running on MacOS could provide an environment for Mac users to develop and run Linux containers on a Mac machine without needing to install Linux via a VM or...

Warning when using an unsupported version of Linux

10

gVisor fails in unintuitive ways when run on an older version of Linux. We should print a warning or simply fail if he version of Linux used is less than...