Rick Dillon rpdillon

Organizations

@apartmentlist
rpdillon deleted branch upstream-prep at apartmentlist/resque-bus
@rpdillon
rpdillon merged pull request apartmentlist/resque-bus#6
@rpdillon
Ensure Resque Redis instance gets restored in all cases
1 commit with 15 additions and 11 deletions
rpdillon commented on pull request queue-bus/resque-bus#3
@rpdillon

We have exactly that use case, and we've written code to do what you describe. That work is not included here because I don't feel that we've naile…

@rpdillon
  • @rpdillon dcfbd4c
    Ensure Resque Redis instance gets restored in all cases
rpdillon opened pull request apartmentlist/resque-bus#6
@rpdillon
Ensure Resque Redis instance gets restored in all cases
1 commit with 14 additions and 10 deletions
rpdillon commented on pull request queue-bus/resque-bus#3
@rpdillon

Sure thing.

rpdillon commented on pull request queue-bus/resque-bus#3
@rpdillon

I think the dangerous method here is the parsing the the assignment does. I'll wrap that and ensure the default gets restored. In the case that the…

@rpdillon
rpdillon commented on pull request queue-bus/resque-bus#3
@rpdillon

No problem...I realized I was doing a bunch of style stuff, which can be obnoxious. I'll extract that from this PR.

rpdillon commented on pull request queue-bus/resque-bus#3
@rpdillon

Great point. I'll fix it.

rpdillon opened pull request queue-bus/resque-bus#3
@rpdillon
Add support for dedicated Redis instance
13 commits with 179 additions and 66 deletions
rpdillon deleted branch bug/fix-queuebus-setup at apartmentlist/resque-bus
@rpdillon
rpdillon merged pull request apartmentlist/resque-bus#5
@rpdillon
Remove extra set_redis_server call in queuebus:setup
1 commit with 1 addition and 1 deletion
rpdillon opened pull request apartmentlist/resque-bus#5
@rpdillon
Remove extra set_redis_server call in queuebus:setup
1 commit with 1 addition and 1 deletion
rpdillon deleted branch debug/troubleshoot-heroku at apartmentlist/resque-bus
@rpdillon
rpdillon merged pull request apartmentlist/resque-bus#4
@rpdillon
Add :use_queuebus_redis to setup task
3 commits with 1 addition and 1 deletion
rpdillon commented on pull request apartmentlist/resque-bus#4
@rpdillon

Nice!

rpdillon deleted branch set-queuebus-server at apartmentlist/resque-bus
@rpdillon
rpdillon merged pull request apartmentlist/resque-bus#3
@rpdillon
Configure QueueBus tasks to use QueueBus Redis
1 commit with 69 additions and 47 deletions
rpdillon commented on pull request apartmentlist/resque-bus#3
@rpdillon

I need to explain how the system works for this to make sense. QUEUEBUS_REDIS_URL needs to be set in the environment of any system that talks to …

rpdillon commented on pull request apartmentlist/resque-bus#3
@rpdillon

Because QUEUEBUS_REDIS_URL will always be set on environments that use QueueBus, regardless of whether the working should actually be working off o…

rpdillon commented on pull request apartmentlist/resque-bus#3
@rpdillon

And finally this.