Github user wy96f commented on the issue:
https://github.com/apache/activemq-artemis/pull/2155
@michaelandrepearce Sorry for the late reply. I rebase the code and tests
pass.
---
Github user michaelandrepearce commented on the issue:
https://github.com/apache/activemq-artemis/pull/2155
@wy96f can you look at the failed build? Ideally need a succesful pr build
to merge
---
Github user wy96f commented on the issue:
https://github.com/apache/activemq-artemis/pull/2155
Test resolved:)
---
Github user jbertram commented on the issue:
https://github.com/apache/activemq-artemis/pull/2155
@wy96f, the test you added has a lot of checkstyle violations. Please
resolve.
---
Github user wy96f commented on the issue:
https://github.com/apache/activemq-artemis/pull/2155
Hi, michaelandrepearce, thanks for review, test has been added to validate
the issue:)
---
Github user michaelandrepearce commented on the issue:
https://github.com/apache/activemq-artemis/pull/2155
Could a test case be added to capture the issue, and therefor also validate
the fix and ensure no future regression?
---
