GitHub user tabish121 opened a pull request:
https://github.com/apache/activemq-artemis/pull/2304
ARTEMIS-2082 Reset buffer valid flag after re-encoding the message
Once a re-encode of the message is done the buffer is not being marked
as valid and so subsequent checks on the buffer are all assuming the
message data is not valid and re-encoding over and over. This can lead
to poor performance in some cases and corrupted data in others.
You can merge this pull request into a Git repository by running:
$ git pull https://github.com/tabish121/activemq-artemis ARTEMIS-2082
Alternatively you can review and apply these changes as the patch at:
https://github.com/apache/activemq-artemis/pull/2304.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 #2304
----
commit e065e3e960ac8cc6505a71ec4f67b58b4ffb5990
Author: Timothy Bish <tabish121@...>
Date: 2018-09-11T19:42:24Z
ARTEMIS-2082 Reset buffer valid flag after re-encoding the message
Once a re-encode of the message is done the buffer is not being marked
as valid and so subsequent checks on the buffer are all assuming the
message data is not valid and re-encoding over and over. This can lead
to poor performance in some cases and corrupted data in others.
----
---
