Github user tabish121 commented on the issue:
https://github.com/apache/activemq-artemis/pull/2031
The change as it is seems acceptable as it does reduce unnecessary
allocations when ByteMessages types cross to OpenWire. I've reviewed the
changes as it relates to the OpenWire objects and the way the converter uses it
and I don't see anything adverse happening from this. As always a run through
the tests would be good for some validation. ---
