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.
---