Github user franz1981 commented on a diff in the pull request:
https://github.com/apache/activemq-artemis/pull/1970#discussion_r176497457
--- Diff:
tests/integration-tests/src/test/java/org/apache/activemq/artemis/tests/integration/client/ConsumerTest.java
---
@@ -446,7 +449,15 @@ public void internalSimpleSend(int protocolSender, int
protocolConsumer) throws
TextMessage msg = session.createTextMessage("hello");
msg.setIntProperty("mycount", 0);
producer.send(msg);
- connection.close();
+
+ StringBuffer bufferLarge = new StringBuffer();
--- End diff --
Would be better to use `StringBuilder` instead, because `StringBuffer` is
`synchronized`
---
