[
https://issues.apache.org/jira/browse/ARTEMIS-2191?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16710710#comment-16710710
]
ASF GitHub Bot commented on ARTEMIS-2191:
-----------------------------------------
Github user clebertsuconic commented on the issue:
https://github.com/apache/activemq-artemis/pull/2449
@franz1981 the failures I mentioned are on my branch.
I am asking you to defer your PR until after I merge mine on AMQP
performance.
This route change I'm making is a bit more functional than yours. It seems
yours is a refactoring while mine had functionality involved.. hence I'm asking
you to defer it
Adding your change in top of mine added failues (the one you sent).
> Makes PostOfficeImpl::route common paths inlineable
> ---------------------------------------------------
>
> Key: ARTEMIS-2191
> URL: https://issues.apache.org/jira/browse/ARTEMIS-2191
> Project: ActiveMQ Artemis
> Issue Type: Improvement
> Components: Broker
> Affects Versions: 2.7.0
> Reporter: Francesco Nigro
> Assignee: Francesco Nigro
> Priority: Minor
>
> PostOfficeImpl::route is handling very difference cases (eg not
> durable/durable messages, not durable/durable queues, transactions etc etc)
> on the same code paths, making the code too big to be inlined when specific
> code paths are needed.
> The code could be refactored to allow better inlining by using smaller
> methods and moving away uncommon/slow path from the common one.
--
This message was sent by Atlassian JIRA
(v7.6.3#76005)
