[ 
https://issues.apache.org/jira/browse/ARTEMIS-3834?focusedWorklogId=773670&page=com.atlassian.jira.plugin.system.issuetabpanels:worklog-tabpanel#worklog-773670
 ]

ASF GitHub Bot logged work on ARTEMIS-3834:
-------------------------------------------

                Author: ASF GitHub Bot
            Created on: 23/May/22 19:05
            Start Date: 23/May/22 19:05
    Worklog Time Spent: 10m 
      Work Description: jbertram commented on PR #4079:
URL: 
https://github.com/apache/activemq-artemis/pull/4079#issuecomment-1135037493

   The real way to fix `QueueControlUsingCoreTest` is just to tell 
`invokeOperation` to cast it to an `Integer.class`, e.g.:
   ```java
            @Override
            public int sendMessagesToDeadLetterAddress(final String filterStr) 
throws Exception {
               return (Integer) 
proxy.invokeOperation(Integer.class,"sendMessagesToDeadLetterAddress", 
filterStr);
            }
   ```
   That said, I'm concerned that this introduces an inconsistency with 
`sendMessageToDeadLetterAddress(long)` which *doesn't* include paged messages. 
I think either both of these methods should work on paged messages or neither 
of them should.
   
   Thoughts?




Issue Time Tracking
-------------------

    Worklog Id:     (was: 773670)
    Time Spent: 1.5h  (was: 1h 20m)

> Include paged messages when running "Send messages to DLA"-job
> --------------------------------------------------------------
>
>                 Key: ARTEMIS-3834
>                 URL: https://issues.apache.org/jira/browse/ARTEMIS-3834
>             Project: ActiveMQ Artemis
>          Issue Type: Improvement
>            Reporter: Anton Roskvist
>            Priority: Major
>          Time Spent: 1.5h
>  Remaining Estimate: 0h
>
> Currently this operation does not include paged messages so it has to be run 
> multiple times



--
This message was sent by Atlassian Jira
(v8.20.7#820007)

Reply via email to