Hi,
Thanks for your contribution !
PR on main branch is enough: we do the review and test on this PR and
then we cherry pick on active branches.
Regards
JB
On 25/01/2022 09:57, Endre Stølsvik wrote:
It is ActiveMQ Classic.
I just went ahead, hoping that it is OK: PR to main branch:
https://github.com/apache/activemq/pull/749
Would it be relevant to also add this as a separate PR to branch
'activemq-5.16.x'? The change is intended to be non-intrusive wrt. existing
behaviour.
Kind regards,
Endre Stølsvik
On Tue, Jan 25, 2022 at 3:04 AM Clebert Suconic <clebert.suco...@gmail.com>
wrote:
Which sub project are you referring to?
A self contained pull request on GitHub with a test case usually suffices
it.
Let me know what sub project and I can point you to the GitHub project and
perhaps some other guides.
On Mon, Jan 24, 2022 at 7:14 PM Endre Stølsvik <en...@stolsvik.com> wrote:
Hi!
I have a small contribution I was hoping that you would consider.
The feature is in the StatisticsBroker(Plugin), where I have added an
optional request property, which if present will on the reply message
include the "BrokerInTime" for the first message in the destination, as a
field "firstMessageTimestamp".
How do you want me to do such a suggestion? Should I first open an issue
/
feature request, and then do a PR for it, or just submit a PR directly?
Kind regards,
Endre Stølsvik
--
Clebert Suconic
