Bogdanp / dramatiq_sqs

A Dramatiq broker that can be used with Amazon SQS.
https://dramatiq.io
Other
56 stars 21 forks source link

[Discussion/Question] why not use sqs visibility timeout to support re-queue rather than creating new messages #16

Open chintan-synapse opened 2 years ago

chintan-synapse commented 2 years ago

@Bogdanp I see you initially skipped implementation of re-queue to use SQS visibility timeout which sounds like a good idea but you later implemented the re-queue logic. Just want to know your thoughts behind this decision.