Github user clebertsuconic commented on the issue:
https://github.com/apache/activemq-artemis/pull/2396
@michaelandrepearce I added a test to encode / decode in multi-thread. it
should be enough.
---
Github user franz1981 commented on the issue:
https://github.com/apache/activemq-artemis/pull/2396
@michaelandrepearce Someone has summoned me? If you repeat my name 3 times
I'll disappear (cit: Beetlejuice)
---
Github user michaelandrepearce commented on the issue:
https://github.com/apache/activemq-artemis/pull/2396
re 2.0.0 and 2.6.x i fully blame @franz198 , little speed deamon! ;) :)
---
Github user michaelandrepearce commented on the issue:
https://github.com/apache/activemq-artemis/pull/2396
re validBuffer im just wondering to simply avoid any possible future
issue all it takes is a year or two's time and we all forget about this and
someone implements
Github user clebertsuconic commented on the issue:
https://github.com/apache/activemq-artemis/pull/2396
about previously from 2.0.0 I can only think that we have made improvements
that probably allowed more load. I ran the same test against 2.0.0 and it fails
as well.
Prior
Github user michaelandrepearce commented on the issue:
https://github.com/apache/activemq-artemis/pull/2396
@clebertsuconic did you see this?
---
Github user michaelandrepearce commented on the issue:
https://github.com/apache/activemq-artemis/pull/2396
Another thing ive noticed is currently where validBuffer is changed its not
protected its only volatile, so you could enter syncronized method do the
validBuffer check, but
Github user clebertsuconic commented on the issue:
https://github.com/apache/activemq-artemis/pull/2396
@michaelandrepearce I will merge this now.. but if you could still review
it after I merged it? we can send further commits.
---
Github user clebertsuconic commented on the issue:
https://github.com/apache/activemq-artemis/pull/2396
This is now ready to be merged.. (I had a word WIP on the name here before);
---
Github user clebertsuconic commented on the issue:
https://github.com/apache/activemq-artemis/pull/2396
@michaelandrepearce can you review this one please.
---
10 matches
Mail list logo