Github user michaelandrepearce commented on the issue:
https://github.com/apache/activemq-artemis/pull/1659
The serialisation change was actually needed for the fix, this is some of
the actual issue.
> On 17 Nov 2017, at 04:17, clebertsuconic <[email protected]> wrote:
>
> The test failure is so weird. if you run the whole thing in Idea it
fails.. individually it's passing.
>
> I can see it's missing a property.. but I'm not sure what changed that
user was at some point considered a property.. as the BeanUtil wasn't picking
up builder pattern on setUser... so weird.
>
> Anyways.. I sent a PR to your branch.. please squash it.
>
> Only thing I'm concerned here is the wire change on the connection
factory (as it was a string with the URI before)... and I'm not so sure if you
really need to move those classes.. thinking this as an API change since
Wildfly will probably need it.
>
> â
> You are receiving this because you were mentioned.
> Reply to this email directly, view it on GitHub
<https://github.com/apache/activemq-artemis/pull/1659#issuecomment-345115665>,
or mute the thread
<https://github.com/notifications/unsubscribe-auth/ABUtLgcu7LPIG2FuPTUm4ALiIxQn_CVrks5s3N6kgaJpZM4QhAPN>.
>
---
