-
Notifications
You must be signed in to change notification settings - Fork 147
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Does this library support message timers #64
Comments
I found there is a branch for this feature but it has never been pulled into main over the past 3 years: |
I had also provided a pull-request, however there does not seem to be a lot of activity from the maintainers (Amazon AWS) to merge this functionality. |
@mihir-aws @robin-aws Guys, can you please look into this. This implementation is currently missing a very basic feature of SQS |
Hi @enVolt - I'm happy to say I was already looking at this and I should be able to resolve it quickly. I had a look at #12 and #40, but I'd actually prefer to support setting this on the SQSMessageProducer instead of the SQSMessage, as that is forwards-compatible with the feature added in JMS 2.0: https://docs.oracle.com/javaee/7/api/javax/jms/JMSProducer.html#setDeliveryDelay-long- Would that meet people's needs well enough? I'm planning on creating my own pull request for this in the next day or two. |
G'day @robin-aws, Yes, that would work well, i.e. allowing the set delay seconds on the SQSMessageProducer. S |
Great! How does this look? |
Robin,
Looks good to me mate.
S
…On Sat, Mar 9, 2019 at 9:06 AM Robin Salkeld ***@***.***> wrote:
Great! How does this look?
#76 <#76>
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
<#64 (comment)>,
or mute the thread
<https://github.com/notifications/unsubscribe-auth/ACsuY24J2fcdMl4gqly4Rdg8LDIr_cS-ks5vUt7WgaJpZM4VOswM>
.
--
-- What the eyes have not seen, what the ears have not heard and what the
heart has not felt... Jannah...
|
Do you have an ETA on when this feature will be released in the master? |
This has been released in 1.0.6 - closing. |
We can set DelaySeconds on individual messages sent to standard sqs queues
https://docs.aws.amazon.com/AWSSimpleQueueService/latest/SQSDeveloperGuide/sqs-send-message-with-timer.html
How to set this parameter using this library
The text was updated successfully, but these errors were encountered: