[GitHub] activemq-artemis pull request #1662: ARTEMIS-1514 Fix read large message int...

Previous Topic Next Topic
 
classic Classic list List threaded Threaded
2 messages Options
Reply | Threaded
Open this post in threaded view
|

[GitHub] activemq-artemis pull request #1662: ARTEMIS-1514 Fix read large message int...

pgfox
GitHub user mtaylor opened a pull request:

    https://github.com/apache/activemq-artemis/pull/1662

    ARTEMIS-1514 Fix read large message into buffer

    @clebertsuconic this fixes your previous commit.  Please take a look.
   
    buffer.toByteBuffer() will generate a byte buffer with capacity set to the number of readable bytes on the buffer.  Since the buffer has no data, the capacity is set to 0.  This causes clients to not get the message body when doing large messages (without streaming).

You can merge this pull request into a Git repository by running:

    $ git pull https://github.com/mtaylor/activemq-artemis ARTEMIS-1514-fix

Alternatively you can review and apply these changes as the patch at:

    https://github.com/apache/activemq-artemis/pull/1662.patch

To close this pull request, make a commit to your master/trunk branch
with (at least) the following in the commit message:

    This closes #1662
   
----
commit eddb144debac2c3dd25e9354151e230e4ab73172
Author: Martyn Taylor <[hidden email]>
Date:   2017-11-21T13:22:27Z

    ARTEMIS-1514 Fix read large message into buffer

----


---
Reply | Threaded
Open this post in threaded view
|

[GitHub] activemq-artemis pull request #1662: ARTEMIS-1514 Fix read large message int...

pgfox
Github user asfgit closed the pull request at:

    https://github.com/apache/activemq-artemis/pull/1662


---