You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@activemq.apache.org by cl...@apache.org on 2019/08/01 20:15:27 UTC

[activemq-artemis] branch master updated (d77edb8 -> 6837150)

This is an automated email from the ASF dual-hosted git repository.

clebertsuconic pushed a change to branch master
in repository https://gitbox.apache.org/repos/asf/activemq-artemis.git.


    from d77edb8  This closes #2739
     new 21af85b  NO-JIRA Speeding up tests
     new b1c3f52  NO-JIRA test fix (Adding Wait clause)
     new 6837150  This closes #2776

The 3 revisions listed above as "new" are entirely new to this
repository and will be described in separate emails.  The revisions
listed as "add" were already present in the repository and have only
been added to this reference.


Summary of changes:
 .../java/org/apache/activemq/artemis/tests/util/ActiveMQTestBase.java | 3 +++
 .../activemq/artemis/tests/integration/amqp/JMSQueueBrowserTest.java  | 4 ++--
 2 files changed, 5 insertions(+), 2 deletions(-)


[activemq-artemis] 02/03: NO-JIRA test fix (Adding Wait clause)

Posted by cl...@apache.org.
This is an automated email from the ASF dual-hosted git repository.

clebertsuconic pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/activemq-artemis.git

commit b1c3f52cc8e3739a6264318e781285cf58a070e1
Author: Clebert Suconic <cl...@apache.org>
AuthorDate: Thu Aug 1 10:01:09 2019 -0400

    NO-JIRA test fix (Adding Wait clause)
---
 .../activemq/artemis/tests/integration/amqp/JMSQueueBrowserTest.java  | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/tests/integration-tests/src/test/java/org/apache/activemq/artemis/tests/integration/amqp/JMSQueueBrowserTest.java b/tests/integration-tests/src/test/java/org/apache/activemq/artemis/tests/integration/amqp/JMSQueueBrowserTest.java
index 0c266e1..7ab5333 100644
--- a/tests/integration-tests/src/test/java/org/apache/activemq/artemis/tests/integration/amqp/JMSQueueBrowserTest.java
+++ b/tests/integration-tests/src/test/java/org/apache/activemq/artemis/tests/integration/amqp/JMSQueueBrowserTest.java
@@ -260,9 +260,9 @@ public class JMSQueueBrowserTest extends JMSClientTestSupport {
       browser.close();
 
       // Now check that all browser work did not affect the session transaction.
-      assertEquals(5, queueView.getMessageCount());
+      Wait.assertEquals(5, queueView::getMessageCount);
       session.commit();
-      assertEquals(10, queueView.getMessageCount());
+      Wait.assertEquals(10, queueView::getMessageCount);
    }
 
    @Test(timeout = 60000)


[activemq-artemis] 03/03: This closes #2776

Posted by cl...@apache.org.
This is an automated email from the ASF dual-hosted git repository.

clebertsuconic pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/activemq-artemis.git

commit 6837150d5ca95b4054de4529f0a017b847ed1dce
Merge: d77edb8 b1c3f52
Author: Clebert Suconic <cl...@apache.org>
AuthorDate: Thu Aug 1 16:15:18 2019 -0400

    This closes #2776

 .../java/org/apache/activemq/artemis/tests/util/ActiveMQTestBase.java | 3 +++
 .../activemq/artemis/tests/integration/amqp/JMSQueueBrowserTest.java  | 4 ++--
 2 files changed, 5 insertions(+), 2 deletions(-)


[activemq-artemis] 01/03: NO-JIRA Speeding up tests

Posted by cl...@apache.org.
This is an automated email from the ASF dual-hosted git repository.

clebertsuconic pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/activemq-artemis.git

commit 21af85bd94ca4612dc1f10f1431963ea99334124
Author: Clebert Suconic <cl...@apache.org>
AuthorDate: Wed Jul 31 12:20:20 2019 -0400

    NO-JIRA Speeding up tests
    
    When it comes to the testsuite, we don't need timed buffers kicking too much.
    This should bring some speed for our testsuite
---
 .../java/org/apache/activemq/artemis/tests/util/ActiveMQTestBase.java  | 3 +++
 1 file changed, 3 insertions(+)

diff --git a/artemis-server/src/test/java/org/apache/activemq/artemis/tests/util/ActiveMQTestBase.java b/artemis-server/src/test/java/org/apache/activemq/artemis/tests/util/ActiveMQTestBase.java
index 22b3173..a318e98 100644
--- a/artemis-server/src/test/java/org/apache/activemq/artemis/tests/util/ActiveMQTestBase.java
+++ b/artemis-server/src/test/java/org/apache/activemq/artemis/tests/util/ActiveMQTestBase.java
@@ -476,6 +476,9 @@ public abstract class ActiveMQTestBase extends Assert {
    protected ConfigurationImpl createBasicConfig(final int serverID) {
       ConfigurationImpl configuration = new ConfigurationImpl().setSecurityEnabled(false).setJournalMinFiles(2).setJournalFileSize(100 * 1024).setJournalType(getDefaultJournalType()).setJournalDirectory(getJournalDir(serverID, false)).setBindingsDirectory(getBindingsDir(serverID, false)).setPagingDirectory(getPageDir(serverID, false)).setLargeMessagesDirectory(getLargeMessagesDir(serverID, false)).setJournalCompactMinFiles(0).setJournalCompactPercentage(0).setClusterPassword(CLUSTER_PASS [...]
 
+      // When it comes to the testsuite, we don't need any batching, I will leave some minimal batching to exercise the codebase
+      configuration.setJournalBufferTimeout_AIO(100).setJournalBufferTimeout_NIO(100);
+
       return configuration;
    }