Add transaction scope around postgres ‘find_by_sql’ statement #79

wants to merge 1 commit into

2 participants


This is in order to prevent multiple instances of DJ from picking up the same task (as the current locking of FOR UPDATE will suffice for a single connection, when scaled out our workers seemed to have been picking up the same job on different machines)


Coverage Status

Changes Unknown when pulling 723ff09 on nimast:feature/postgres-reserve-transaction into ** on collectiveidea:master**.

@unionsmarket unionsmarket added a commit to unionsmarket/delayed_job_active_record that referenced this pull request Jan 16, 2014
@unionsmarket unionsmarket Fix PG bug e6e049b
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment