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

Posted to dev@activemq.apache.org by clebertsuconic <gi...@git.apache.org> on 2017/03/06 04:01:14 UTC

[GitHub] activemq-artemis pull request #1064: Pure message encoding on artemis

GitHub user clebertsuconic opened a pull request:

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

    Pure message encoding on artemis

    As previously discussed, this will allow messages to be transfered on their original encoding.
    
    I foresee a few tests failing due to the current dependency on converters. but I fixed as many as I could... i only increased the number of failures by a handful.. pretty close to what we have now...I think it's worth to merge this and continue to work on master.

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

    $ git pull https://github.com/clebertsuconic/activemq-artemis artemis-1009

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

    https://github.com/apache/activemq-artemis/pull/1064.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 #1064
    
----
commit c0fe187666c0082b8491b1d282cdc93ddc7cdb1e
Author: Clebert Suconic <cl...@apache.org>
Date:   2017-02-07T00:57:16Z

    ARTEMIS-1019 removing vertx integration

commit c1fa5d07c78d198f2cefdfba3674227acad82081
Author: Clebert Suconic <cl...@apache.org>
Date:   2017-02-07T02:17:39Z

    ARTEMIS-1019 Removing Aeroegear
    
    Aeroegear has other ways to use Artemis

commit 669e7cf2a5d460bdda1d8522b129d37ae4517288
Author: Clebert Suconic <cl...@apache.org>
Date:   2017-02-20T20:55:15Z

    ARTEMIS-1009 Pure Message Encoding.
    
    with this we could send and receive message in their raw format,
    without requiring conversions to Core.
    
    - MessageImpl and ServerMessage are removed as part of this
    - AMQPMessage and CoreMessage will have the specialized message format for each protocol
    - The protocol manager is now responsible to send the message
    - The message will provide an encoder for journal and paging

----


---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastructure@apache.org or file a JIRA ticket
with INFRA.
---

[GitHub] activemq-artemis pull request #1064: Pure message encoding on artemis

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

Github user asfgit closed the pull request at:

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


---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastructure@apache.org or file a JIRA ticket
with INFRA.
---

[GitHub] activemq-artemis issue #1064: Pure message encoding on artemis

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

Github user mtaylor commented on the issue:

    https://github.com/apache/activemq-artemis/pull/1064
  
    @clebertsuconic Great work Clebert.
    
    Full test suite looks good and I've done some sanity checking and cross protocol checks, all works swimmingly.
    
    This is a mammoth amount of work @clebertsuconic and the resultant model is much cleaner and I can see we're getting really great perf gains.  Muchos Kudos!!!
    
    Merging.


---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastructure@apache.org or file a JIRA ticket
with INFRA.
---