Github user franz1981 commented on the issue:

    https://github.com/apache/activemq-artemis/pull/1650
  
    @clebertsuconic I've just pushed a PR to your version with very few changes 
(but sadly seems more using diff on the code) mostly perf dependent and to make 
the code a lil more documented: I'm happy with your changes and the added test 
too, very well done!!!!! :+1: 
    
    Let me know if the changes I've pushed seems good considering what are your 
expectations on the mechanics of the final version.
    
    Just a note (if possible): now we have some sanity tests on the 
OrderedExecutor hence I will be super happy to see the missing ones for the 
other critical methods, if any (eg: flush, shutdown)..if is not possible we'll 
add them later.


---

Reply via email to