alphagov / trade-tariff-backend

Enabling the population and distribution via API of UK Customs tariffs and duties
MIT License
7 stars 6 forks source link

Switch to redis-semaphore #164

Closed rodzyn closed 10 years ago

jabley commented 10 years ago

Hello, welcome.

We have published some guidance on how we like our commit messages. Please could you amend your commit accordingly.

Thanks.

rodzyn commented 10 years ago

Hello, thanks for pointing out. I hope current commit message is more descriptive.

matthewford commented 10 years ago

I think additionally the semaphore does not have a timeout, where redis-lock does. This is better because sometimes a sync task could be longer than 30 mins, in the case of a rollback for example.