You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@activemq.apache.org by ro...@apache.org on 2022/06/17 15:26:51 UTC

[activemq-artemis] branch new-logging updated: some more fixups following review of delta on main prior to last rebase, from e49a9521aaf6b706d692d797c32100a205704186 to d0d2a9a16bb6664d1ebeb977f7213a9b8fa640da

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

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


The following commit(s) were added to refs/heads/new-logging by this push:
     new 6c8f3a257c some more fixups following review of delta on main prior to last rebase, from e49a9521aaf6b706d692d797c32100a205704186 to d0d2a9a16bb6664d1ebeb977f7213a9b8fa640da
6c8f3a257c is described below

commit 6c8f3a257c07b3a7e7835b3ba8208cf66d7c1e8c
Author: Robbie Gemmell <ro...@apache.org>
AuthorDate: Fri Jun 17 16:25:23 2022 +0100

    some more fixups following review of delta on main prior to last rebase, from
    e49a9521aaf6b706d692d797c32100a205704186 to d0d2a9a16bb6664d1ebeb977f7213a9b8fa640da
---
 .../artemis/tests/unit/core/paging/impl/PagingStoreImplTest.java  | 8 ++++----
 1 file changed, 4 insertions(+), 4 deletions(-)

diff --git a/tests/unit-tests/src/test/java/org/apache/activemq/artemis/tests/unit/core/paging/impl/PagingStoreImplTest.java b/tests/unit-tests/src/test/java/org/apache/activemq/artemis/tests/unit/core/paging/impl/PagingStoreImplTest.java
index 80ce954b8a..4acd8dd2aa 100644
--- a/tests/unit-tests/src/test/java/org/apache/activemq/artemis/tests/unit/core/paging/impl/PagingStoreImplTest.java
+++ b/tests/unit-tests/src/test/java/org/apache/activemq/artemis/tests/unit/core/paging/impl/PagingStoreImplTest.java
@@ -280,7 +280,7 @@ public class PagingStoreImplTest extends ActiveMQTestBase {
 
       for (int repeat = 0; repeat < 5; repeat++) {
          log.debug("###############################################################################################################################");
-         log.debug("#repeat " + repeat);
+         log.debug("#repeat {}", repeat);
 
          storeImpl.startPaging();
          Assert.assertEquals(1, storeImpl.getNumberOfPages());
@@ -340,7 +340,7 @@ public class PagingStoreImplTest extends ActiveMQTestBase {
 
             Assert.assertTrue(subscription.contains(reference));
 
-            log.debug("#received message " + messagesRead + ", " + reference);
+            log.debug("#received message {}, {}", messagesRead, reference);
             messagesRead++;
             int pageOnMsg = reference.getMessage().getIntProperty("page");
             Assert.assertTrue("received " + reference, pageOnMsg <= 2 || pageOnMsg >= 10);
@@ -367,7 +367,7 @@ public class PagingStoreImplTest extends ActiveMQTestBase {
       for (long pgID = startPage; pgID <= endPage; pgID++) {
          Page page = storeImpl.newPageObject(pgID);
          page.open(false);
-         log.debug("# Page " + pgID);
+         log.debug("# Page {}", pgID);
          page.getMessages().forEach(p -> {
             String acked;
             try {
@@ -376,7 +376,7 @@ public class PagingStoreImplTest extends ActiveMQTestBase {
                e.printStackTrace();
                acked = "";
             }
-            log.debug(acked + p);
+            log.debug("{}{}", acked, p);
          });
          page.close(false);
       }