Add Documentation about all options of Concurrency about activejob-traffic_control HOT 4 OPEN

@nickelser one important feature, for which docs are missing is the key attribute in the options for concurrency. After reading through the code, I realized that it can be a Proc/Lamda, which totally worked for my use case (well, to a point, but the gem works as it should).

I did experience a problem with a Sidekiq backed job, which had named (not positional) parameters. The job would be called seemingly w/o parameters. I did change the signature of the perform method and got around it. Perhaps I should open another issue?

from activejob-traffic_control.

@psteininger I did try to document most things, sorry for the confusion!

I would love a PR, but barring that -- any details on what specifically was missing would be appreciated and I am happy to update!

from activejob-traffic_control.

@psteininger makes total sense -- I will add that to the docs. I also created a new issue to track the named arguments problem (#5), which I'll look at shortly (could definitely be an issue how we re-enqueue the jobs).

from activejob-traffic_control.

As with @nickelser, I had to read the source (and this issue) to realize the options for the key parameter. I see the following options for Concurrency

key:           nil,
wait_timeout:  0.1, 
stale_timeout: 60 * 10

I think it would be helpful to document all of these. Either in rdoc or in the README (or both).

from activejob-traffic_control.