git.net

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

[GitHub] activemq-artemis pull request #1722: (WIP DO NOT MERGE) ARTEMIS-1545 Support...


Github user franz1981 commented on a diff in the pull request:

    https://github.com/apache/activemq-artemis/pull/1722#discussion_r157485298
  
    --- Diff: artemis-server/src/main/java/org/apache/activemq/artemis/core/protocol/ServerPacketDecoder.java ---
    @@ -115,6 +123,8 @@ public Packet decode(final ActiveMQBuffer in) {
           switch (packetType) {
              case SESS_SEND:
                 return decodeSessionSendMessage(in);
    +         case SESS_SEND_V2:
    +            return decodeSessionSendMessageV2(in);
    --- End diff --
    
    If decodeSessionSendMessageV2 is not supposed to be the common case here, would be better to move it on the `slowPathDecode`  instead


---



( ! ) Warning: include(msgfooter.php): failed to open stream: No such file or directory in /var/www/git/apache-activemq-developers/msg08202.html on line 86
Call Stack
#TimeMemoryFunctionLocation
10.0009363704{main}( ).../msg08202.html:0

( ! ) Warning: include(): Failed opening 'msgfooter.php' for inclusion (include_path='.:/var/www/git') in /var/www/git/apache-activemq-developers/msg08202.html on line 86
Call Stack
#TimeMemoryFunctionLocation
10.0009363704{main}( ).../msg08202.html:0