You are viewing a plain text version of this content. The canonical link for it is here.

Posted to dev@activemq.apache.org by tabish121 <gi...@git.apache.org> on 2018/09/25 16:39:21 UTC

[GitHub] activemq-artemis pull request #2329: ARTEMIS-2096 Refactor AMQMessage abstra...

GitHub user tabish121 opened a pull request:

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

    ARTEMIS-2096 Refactor AMQMessage abstraction

    Major refactoring of the AMQPMessage abstraction to resolve
    some issue of message corruption still present in the code and
    improve the API handling of message changes and re-encoding.
    
    Improves handling of decoding of message sections limiting the
    work to only the portions needed and ensuring the state data
    is always updated with what has been done.  Fixes issues of
    corrupt state on copy of message or other changes in filters.

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

    $ git pull https://github.com/tabish121/activemq-artemis ARTEMIS-2096

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

    https://github.com/apache/activemq-artemis/pull/2329.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 #2329
    
----
commit ed7e7f206287db56f0c44c01fa2133c1fbd5c50d
Author: Timothy Bish <ta...@...>
Date:   2018-09-25T16:22:19Z

    ARTEMIS-2096 Refactor AMQMessage abstraction
    
    Major refactoring of the AMQPMessage abstraction to resolve
    some issue of message corruption still present in the code and
    improve the API handling of message changes and re-encoding.
    
    Improves handling of decoding of message sections limiting the
    work to only the portions needed and ensuring the state data
    is always updated with what has been done.  Fixes issues of
    corrupt state on copy of message or other changes in filters.

----


---

[GitHub] activemq-artemis issue #2329: ARTEMIS-2096 Refactor AMQMessage abstraction

Posted by clebertsuconic <gi...@git.apache.org>.

Github user clebertsuconic commented on the issue:

    https://github.com/apache/activemq-artemis/pull/2329
  
    I hope this is not 2.6.x


---

[GitHub] activemq-artemis issue #2329: ARTEMIS-2096 Refactor AMQMessage abstraction

Posted by clebertsuconic <gi...@git.apache.org>.

Github user clebertsuconic commented on the issue:

    https://github.com/apache/activemq-artemis/pull/2329
  
    I will run a full testsuite before merging this, given the number of line changes.


---

[GitHub] activemq-artemis pull request #2329: ARTEMIS-2096 Refactor AMQMessage abstra...

Posted by asfgit <gi...@git.apache.org>.

Github user asfgit closed the pull request at:

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


---