activemq-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From cshannon <>
Subject [GitHub] activemq-artemis pull request #1816: ARTEMIS-1633 - fire message routing cal...
Date Thu, 25 Jan 2018 18:16:30 GMT
Github user cshannon commented on a diff in the pull request:
    --- Diff: artemis-server/src/main/java/org/apache/activemq/artemis/core/postoffice/impl/
    @@ -772,6 +772,8 @@ public RoutingStatus route(final Message message,
    +      server.callBrokerPlugins(server.hasBrokerPlugins() ? plugin -> plugin.beforeMessageRoute(message,
context, direct, rejectDuplicates) : null);
    --- End diff --
    If there were messages sent to an address with no bindings the message would get dropped
because of lack of bindings.  But with the old way the message route methods were only triggered
if the routing status was OK.  With the change the methods are called even if there are no
bindings but the status will say NO_BINDINGS, etc.


View raw message