[
https://issues.apache.org/jira/browse/ARTEMIS-3909?focusedWorklogId=796137&page=com.atlassian.jira.plugin.system.issuetabpanels:worklog-tabpanel#worklog-796137
]
ASF GitHub Bot logged work on ARTEMIS-3909:
-------------------------------------------
Author: ASF GitHub Bot
Created on: 28/Jul/22 17:04
Start Date: 28/Jul/22 17:04
Worklog Time Spent: 10m
Work Description: clebertsuconic merged PR #4156:
URL: https://github.com/apache/activemq-artemis/pull/4156
Issue Time Tracking
-------------------
Worklog Id: (was: 796137)
Time Spent: 20m (was: 10m)
> Move RoutingContext::processReferences to PostOfficeImpl
> --------------------------------------------------------
>
> Key: ARTEMIS-3909
> URL: https://issues.apache.org/jira/browse/ARTEMIS-3909
> Project: ActiveMQ Artemis
> Issue Type: Task
> Reporter: Clebert Suconic
> Assignee: Clebert Suconic
> Priority: Major
> Fix For: 2.25.0
>
> Time Spent: 20m
> Remaining Estimate: 0h
>
> Reasoning:
> - We reuse the Routing Context as an optimization on the ServerSession.
> - By the time the Routing is done, we will process the references on a IO
> callback.
> - if anyone used any state in between the Routing and Storage, you could have
> an invalid operation.
> We were lucky that the method processReferences in PostOffice was pretty much
> static and no actual bug happened because of that, however I almost thought
> we had something scary going on.
> So, to avoid this kind of issue ever being introduced in the codebase I am
> moving the processReferences away from RoutingContext.
--
This message was sent by Atlassian Jira
(v8.20.10#820010)
