activemq-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From calohmn <>
Subject [GitHub] activemq-artemis pull request #2256: ARTEMIS-2045 Add support for setting de...
Date Tue, 21 Aug 2018 11:49:05 GMT
Github user calohmn commented on a diff in the pull request:
    --- Diff: artemis-protocols/artemis-amqp-protocol/src/main/java/org/apache/activemq/artemis/protocol/amqp/broker/
    @@ -733,6 +743,9 @@ public void sendBuffer(ByteBuf buffer, int deliveryCount) {
    +      writeOutgoingDeliveryAnnotations(buffer);
    --- End diff --
    I'm not sure here:
    This addition here (in `sendBuffer(buffer, deliveryCount)` from the `Message` interface)
was done to have both the `sendBuffer` and `getSendBuffer` methods behave the same towards
using the `outgoingDeliveryAnnotations`.
    But on the other hand this `sendBuffer` method isn't called by the `ProtonServerSenderContext`
(only `getSendBuffer` is).
    So maybe the better solution would be to remove usage of `outgoingDeliveryAnnotations`
in the `sendBuffer` method here and rename `getSendBuffer` to `getOutgoingMessageSendBuffer`
to avoid confusion concerning these methods.


View raw message