[
https://issues.apache.org/jira/browse/AMQ-9646?focusedWorklogId=954416&page=com.atlassian.jira.plugin.system.issuetabpanels:worklog-tabpanel#worklog-954416
]
ASF GitHub Bot logged work on AMQ-9646:
---------------------------------------
Author: ASF GitHub Bot
Created on: 27/Jan/25 21:23
Start Date: 27/Jan/25 21:23
Worklog Time Spent: 10m
Work Description: cshannon commented on PR #1377:
URL: https://github.com/apache/activemq/pull/1377#issuecomment-2616923918
> > static class MessageRecoveryConfig {
> > final String startMsgId;
> > ...
>
> I was thinking a MessageRecoveryContext would make sense where it could be
an extension of the listener w/ config options.
>
> Then the method could take one parameter: MessageRecoveryContext ..
>
> ```
> static class MessageRecoveryContext {
>
> final MessageRecoveryListener listener;
> final long offset;
> final String startMsgId ..
> ```
>
> However, this also introduces complexity of deciding which fields to use..
the offset+maxRecovered and/or the startMsgId and/or the endMsgId.. etc.
Yeah, you have to get into validation in that case. Really the question is
do you plan to keep expanding the features? I don't think we want to get into a
situation where we have 20 different overloaded methods with different argument
combinations. The more flags we have the better it is to just make some sort of
configuration type object and then just validate the flag combinations that are
set are correct.
Issue Time Tracking
-------------------
Worklog Id: (was: 954416)
Time Spent: 2h 10m (was: 2h)
> Support selecting specific messages for command line backup
> -----------------------------------------------------------
>
> Key: AMQ-9646
> URL: https://issues.apache.org/jira/browse/AMQ-9646
> Project: ActiveMQ Classic
> Issue Type: Improvement
> Reporter: Matt Pavlovich
> Assignee: Matt Pavlovich
> Priority: Minor
> Fix For: 6.2.0, 5.19.0
>
> Time Spent: 2h 10m
> Remaining Estimate: 0h
>
--
This message was sent by Atlassian Jira
(v8.20.10#820010)
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
For further information, visit: https://activemq.apache.org/contact
