[jboss-jira] [JBoss JIRA] Commented: (JBMESSAGING-1203) Re-enable scheduled delivery

[Tim Fox (JIRA)]

    [ https://jira.jboss.org/jira/browse/JBMESSAGING-1203?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=12432152#action_12432152 ] 

Tim Fox commented on JBMESSAGING-1203:
--------------------------------------

I don't like the idea of putting it on the message since this is not a major use case and we don't want to add extra size to the message for the 99.99% of cases that won't use it.

You don't need it on the message to be able to persist the information. Just add an update record for the scheduled delivery time where appropriate.

If you look in JournalStorageManager there's already a TODO for this:

case SET_SCHEDULED_DELIVERY_TIME:
            {
               // TODO
            }

And the constant already exists

> Re-enable scheduled delivery
> ----------------------------
>
>                 Key: JBMESSAGING-1203
>                 URL: https://jira.jboss.org/jira/browse/JBMESSAGING-1203
>             Project: JBoss Messaging
>          Issue Type: Task
>            Reporter: Tim Fox
>            Assignee: Andy Taylor
>             Fix For: 2.0.0 Beta
>
>
> Once we have decided whether scheduled delivery should be an attribute on the messages or an attribute on the send packet. (Probably send packet)

-- 
This message is automatically generated by JIRA.
-
If you think it was sent incorrectly contact one of the administrators: https://jira.jboss.org/jira/secure/Administrators.jspa
-
For more information on JIRA, see: http://www.atlassian.com/software/jira