activemq-commits mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From tab...@apache.org
Subject activemq git commit: Shorten the test a bit with tigther memory limits which reduces the amount of messages we need to send to fill and trigger plist store use. Also add some time to the timeout to account for very slow disk I/O.
Date Tue, 28 Apr 2015 16:19:47 GMT
Repository: activemq
Updated Branches:
  refs/heads/master f5283a904 -> d2248e92d


Shorten the test a bit with tigther memory limits which reduces the
amount of messages we need to send to fill and trigger plist store use.
Also add some time to the timeout to account for very slow disk I/O.

Project: http://git-wip-us.apache.org/repos/asf/activemq/repo
Commit: http://git-wip-us.apache.org/repos/asf/activemq/commit/d2248e92
Tree: http://git-wip-us.apache.org/repos/asf/activemq/tree/d2248e92
Diff: http://git-wip-us.apache.org/repos/asf/activemq/diff/d2248e92

Branch: refs/heads/master
Commit: d2248e92d7236617c3d50f09bfbef939a6b8009e
Parents: f5283a9
Author: Timothy Bish <tabish121@gmail.com>
Authored: Tue Apr 28 12:19:37 2015 -0400
Committer: Timothy Bish <tabish121@gmail.com>
Committed: Tue Apr 28 12:19:37 2015 -0400

----------------------------------------------------------------------
 .../transport/stomp/StompVirtualTopicTest.java  | 61 ++++++++++----------
 1 file changed, 30 insertions(+), 31 deletions(-)
----------------------------------------------------------------------


http://git-wip-us.apache.org/repos/asf/activemq/blob/d2248e92/activemq-stomp/src/test/java/org/apache/activemq/transport/stomp/StompVirtualTopicTest.java
----------------------------------------------------------------------
diff --git a/activemq-stomp/src/test/java/org/apache/activemq/transport/stomp/StompVirtualTopicTest.java
b/activemq-stomp/src/test/java/org/apache/activemq/transport/stomp/StompVirtualTopicTest.java
index ab5069a..cfa28b5 100644
--- a/activemq-stomp/src/test/java/org/apache/activemq/transport/stomp/StompVirtualTopicTest.java
+++ b/activemq-stomp/src/test/java/org/apache/activemq/transport/stomp/StompVirtualTopicTest.java
@@ -27,6 +27,7 @@ import java.util.HashMap;
 import java.util.HashSet;
 import java.util.List;
 import java.util.concurrent.CountDownLatch;
+import java.util.concurrent.TimeUnit;
 
 import javax.management.ObjectName;
 
@@ -44,7 +45,7 @@ import org.slf4j.LoggerFactory;
 public class StompVirtualTopicTest extends StompTestSupport {
 
     private static final Logger LOG = LoggerFactory.getLogger(StompVirtualTopicTest.class);
-    private static final int NUM_MSGS = 100000;
+    private static final int NUM_MSGS = 30000;
 
     private String failMsg = null;
 
@@ -75,7 +76,7 @@ public class StompVirtualTopicTest extends StompTestSupport {
         final PolicyEntry entry = new PolicyEntry();
         entry.setQueue(">");
         entry.setProducerFlowControl(false);
-        entry.setMemoryLimit(10485760);
+        entry.setMemoryLimit(262144);
         entry.setPendingQueuePolicy(new FilePendingQueueMessageStoragePolicy());
         policyEntries.add(entry);
 
@@ -84,13 +85,18 @@ public class StompVirtualTopicTest extends StompTestSupport {
         brokerService.setDestinationPolicy(policyMap);
     }
 
-    @Test(timeout = 60000)
+    @Test(timeout = 90000)
     public void testStompOnVirtualTopics() throws Exception {
         LOG.info("Running Stomp Producer");
 
         StompConsumer consumerWorker = new StompConsumer(this);
         Thread consumer = new Thread(consumerWorker);
 
+        StringBuilder payload = new StringBuilder();
+        for (int i = 0; i < 128; ++i) {
+            payload.append('*');
+        }
+
         consumer.start();
         consumerWorker.awaitStartCompleted();
         Thread.sleep(500);
@@ -100,22 +106,21 @@ public class StompVirtualTopicTest extends StompTestSupport {
         assertTrue(frame.toString().startsWith("CONNECTED"));
 
         for (int i = 0; i < NUM_MSGS - 1; i++) {
-            stompConnection.send("/topic/VirtualTopic.FOO", "Hello World {" + (i + 1) + "}");
+            stompConnection.send("/topic/VirtualTopic.FOO", "Hello World {" + (i + 1) + "}
" + payload.toString());
         }
 
         LOG.info("Sending last packet with receipt header");
         HashMap<String, Object> headers = new HashMap<String, Object>();
         headers.put("receipt", "1234");
         stompConnection.appendHeaders(headers);
-        String msg = "SEND\n" + "destination:/topic/VirtualTopic.FOO\n" +
-                     "receipt: msg-1\n" + "\n\n" + "Hello World {" + (NUM_MSGS-1) + "}" +
Stomp.NULL;
+        String msg = "SEND\n" + "destination:/topic/VirtualTopic.FOO\n" + "receipt: msg-1\n"
+ "\n\n" + "Hello World {" + (NUM_MSGS - 1) + "}" + Stomp.NULL;
         stompConnection.sendFrame(msg);
 
         msg = stompConnection.receiveFrame();
         assertTrue(msg.contains("RECEIPT"));
 
         stompConnection.disconnect();
-        Thread.sleep(1000);
+        TimeUnit.MILLISECONDS.sleep(100);
         stompConnection.close();
         LOG.info("Stomp Producer finished. Waiting for consumer to join.");
 
@@ -131,8 +136,8 @@ public class StompVirtualTopicTest extends StompTestSupport {
     }
 
     /*
-     * Allow Consumer thread to indicate the test has failed.
-     * JUnits Assert.fail() does not work in threads spawned.
+     * Allow Consumer thread to indicate the test has failed. JUnits
+     * Assert.fail() does not work in threads spawned.
      */
     protected void setFail(String msg) {
         this.failMsg = msg;
@@ -180,10 +185,10 @@ public class StompVirtualTopicTest extends StompTestSupport {
 
                 latch.countDown();
 
-                for (counter=0; counter<StompVirtualTopicTest.NUM_MSGS; counter++) {
+                for (counter = 0; counter < StompVirtualTopicTest.NUM_MSGS; counter++)
{
                     frame = stompConnection.receive(15000);
                     log.trace("Received msg with content: " + frame.getBody());
-                    if(!received.add(frame.getBody())) {
+                    if (!received.add(frame.getBody())) {
                         dups.add(frame.getBody());
                     }
                 }
@@ -193,8 +198,7 @@ public class StompVirtualTopicTest extends StompTestSupport {
                     frame = stompConnection.receive(3000);
                     assertNull(frame);
                 } catch (Exception e) {
-                    LOG.info("Correctly received " + e + " while trying to consume an additional
msg." +
-                            " This is expected as the queue should be empty now.");
+                    LOG.info("Correctly received " + e + " while trying to consume an additional
msg." + " This is expected as the queue should be empty now.");
                 }
 
                 // in addition check QueueSize using JMX
@@ -204,16 +208,15 @@ public class StompVirtualTopicTest extends StompTestSupport {
                     parent.setFail("QueueSize not 0 after test has finished.");
                 }
 
-                log.debug("Stomp Consumer Received " + counter + " of " + StompVirtualTopicTest.NUM_MSGS
+
-                          " messages. Check QueueSize in JMX and try to browse the queue.");
+                log.debug("Stomp Consumer Received " + counter + " of " + StompVirtualTopicTest.NUM_MSGS
+                    + " messages. Check QueueSize in JMX and try to browse the queue.");
 
-                if(!dups.isEmpty()) {
-                    for(String msg : dups) {
+                if (!dups.isEmpty()) {
+                    for (String msg : dups) {
                         LOG.debug("Received duplicate message: " + msg);
                     }
 
-                    parent.setFail("Received " + StompVirtualTopicTest.NUM_MSGS +
-                                   " messages but " + dups.size() + " were dups.");
+                    parent.setFail("Received " + StompVirtualTopicTest.NUM_MSGS + " messages
but " + dups.size() + " were dups.");
                 }
 
             } catch (Exception ex) {
@@ -224,13 +227,13 @@ public class StompVirtualTopicTest extends StompTestSupport {
                 } catch (Exception e) {
                 }
 
-                parent.setFail("Stomp Consumer received " + counter + " of " + StompVirtualTopicTest.NUM_MSGS
+
-                               " messages. Check QueueSize in JMX and try to browse the queue.");
+                parent.setFail("Stomp Consumer received " + counter + " of " + StompVirtualTopicTest.NUM_MSGS
+                    + " messages. Check QueueSize in JMX and try to browse the queue.");
 
             } finally {
                 try {
                     stompConnection.disconnect();
-                    Thread.sleep(1000);
+                    TimeUnit.MILLISECONDS.sleep(100);
                     stompConnection.close();
                 } catch (Exception e) {
                     log.error("unexpected exception on sleep", e);
@@ -242,18 +245,14 @@ public class StompVirtualTopicTest extends StompTestSupport {
 
         private long reportQueueStatistics() throws Exception {
 
-            ObjectName queueViewMBeanName = new ObjectName("org.apache.activemq:destinationType=Queue"
+
-                                                           ",destinationName=Consumer.A.VirtualTopic.FOO"
+
-                                                           ",type=Broker,brokerName=localhost");
-            QueueViewMBean queue = (QueueViewMBean)
-                brokerService.getManagementContext().newProxyInstance(queueViewMBeanName,
QueueViewMBean.class, true);
+            ObjectName queueViewMBeanName = new ObjectName("org.apache.activemq:destinationType=Queue"
+ ",destinationName=Consumer.A.VirtualTopic.FOO"
+                + ",type=Broker,brokerName=localhost");
+            QueueViewMBean queue = (QueueViewMBean) brokerService.getManagementContext().newProxyInstance(queueViewMBeanName,
QueueViewMBean.class, true);
 
-            LOG.info("Consumer.A.VirtualTopic.FOO Inflight: " + queue.getInFlightCount()
+
-                     ", enqueueCount: " + queue.getEnqueueCount() + ", dequeueCount: " +
-                     queue.getDequeueCount() + ", dispatchCount: " + queue.getDispatchCount());
+            LOG.info("Consumer.A.VirtualTopic.FOO Inflight: " + queue.getInFlightCount()
+ ", enqueueCount: " + queue.getEnqueueCount() + ", dequeueCount: "
+                + queue.getDequeueCount() + ", dispatchCount: " + queue.getDispatchCount());
 
             return queue.getQueueSize();
         }
     }
 }
-


Mime
View raw message