Github user franz1981 commented on the issue:

    https://github.com/apache/activemq-artemis/pull/1793
  
    @clebertsuconic @RaiSaurabh @michaelandrepearce 
    **Re performances**
    I've the first results using just a small benchmark with 1 producer/1 
consumer using 100 bytes messages.
    ```
    this PR:    37050 msg/sec
    2.5:        85689 msg/sec
    ```
    So performance-wise there is a long way to go, but I think it is related to 
some correctness issues.
    
    **Re correctness**
    I've launched a run on CI just with the OpenWire test suite with no success:
    ```
    [INFO] ActiveMQ5.x unit tests ............................. FAILURE [  
2.950 s]
    ```
    So I've decided to run the most some on my box@home.
    The results from `org.apache.activemq.artemis.tests.integration.openwire` 
ones shows 7 failed tests/341:
    ```
    CompressedInteropTest.testCoreReceiveOpenWireCompressedMessages
    MessageListenerRedeliveryTest.testQueueSessionListenerExceptionDlq
    OpenWireLargeMessageTest.testSendReceiveLargeMessage
    GeneralInteropTest.testSendingToCoreJms
    SimpleOpenWireTest.testTempQueueSendAfterConnectionClose
    RedeliveryPolicyTest.testRedeliveryPolicyPerDestination
    RedeliveryPolicyTest.testDLQHandling
    ```
    I've to run the `tests/activemq5-unit-tests` yet but I suppose that's 
enough to be fixed.
    Sorry for letting this PR sitting here for such a long time and thanks 
@RaiSaurabh to have worked on it: I think that given the right direction taken 
by the work on this PR makes a lot of sense to fix all the compatibility issues 
first, before looking to the performance gains that in the current state, are 
not present. @RaiSaurabh don't give up with the hard work done here and take a 
look at the tests results: as much as possible I will try to help by reviewing 
the code and propose fixes :+1: 



---

Reply via email to