[JMS_SPEC-148] The Delivery Delay Feature Should Utilize the Java SE 8 Date/Time API Created: 02/Apr/14  Updated: 13/Jun/14

Status: Reopened
Project: jms-spec
Component/s: None
Affects Version/s: None
Fix Version/s: None

Type: Improvement Priority: Minor
Reporter: reza_rahman Assignee: Unassigned
Resolution: Unresolved Votes: 0
Labels: None
Remaining Estimate: Not Specified
Time Spent: Not Specified
Original Estimate: Not Specified

Tags: jms21-forreview-minor


Isn't a much better fit for this purpose than the older Java SE APIs.

Comment by Nigel Deakin [ 09/May/14 ]

Please state what you are proposing.

In JMS 2.0, an application requests a delivery delay by specifying a minimum time interval in milliseconds. It doesn't involve the use of "the older Java SE APIs".

Comment by Nigel Deakin [ 14/May/14 ]

More information is needed to be able to assess this request (and I don't want to start guessing at what you had in mind). So I'm marking this as "incomplete".

Will re-open if specific proposals are made.

Comment by reza_rahman [ 03/Jun/14 ]

Returning back to this, hopefully it's not too late...

As I had noted (and you did as well), this is a relatively minor issue as JMSProducer.setDeliveryDelay uses long as an argument type as opposed to the old temporal APIs. However, this feature can arguably be made more type safe/easy to use by allowing an argument of one of the newer Java SE 8 types such as Period or Duration: http://docs.oracle.com/javase/tutorial/datetime/iso/overview.html. For one, that would make any date/time manipulation required before invoking setDeliveryDelay far easier. On the flip side the advantage of the current signature is that it is rather generic (basically down to the lowest common temporal denominator).

Does this help any?

Comment by Nigel Deakin [ 13/Jun/14 ]

Reopening in the light of more information, and tagging for discussion.

Comment by Nigel Deakin [ 13/Jun/14 ]

If we decide to add new methods to set/get delivery delay then we should add similar methods to set time to live.

Generated at Sat Sep 05 17:56:37 UTC 2015 using JIRA 6.2.3#6260-sha1:63ef1d6dac3f4f4d7db4c1effd405ba38ccdc558.