[activemq-artemis] branch main updated: ARTEMIS-4247 Fixing BrokerInSyncTest::testExpiryNoReaper

[clebertsuconic]

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

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


The following commit(s) were added to refs/heads/main by this push:
     new c2ad54fa91 ARTEMIS-4247 Fixing BrokerInSyncTest::testExpiryNoReaper
c2ad54fa91 is described below

commit c2ad54fa917a92ee8acfe701be9201ce30baa1fa
Author: Clebert Suconic <clebertsuco...@apache.org>
AuthorDate: Wed Apr 26 00:28:10 2023 -0400

    ARTEMIS-4247 Fixing BrokerInSyncTest::testExpiryNoReaper
    
    Once the context is set to disable mirror, it should stay that way even if 
reset is called.
---
 .../activemq/artemis/core/server/impl/RoutingContextImpl.java      | 7 ++++++-
 1 file changed, 6 insertions(+), 1 deletion(-)

diff --git 
a/artemis-server/src/main/java/org/apache/activemq/artemis/core/server/impl/RoutingContextImpl.java
 
b/artemis-server/src/main/java/org/apache/activemq/artemis/core/server/impl/RoutingContextImpl.java
index 8373bc2586..d4bcfcebe5 100644
--- 
a/artemis-server/src/main/java/org/apache/activemq/artemis/core/server/impl/RoutingContextImpl.java
+++ 
b/artemis-server/src/main/java/org/apache/activemq/artemis/core/server/impl/RoutingContextImpl.java
@@ -167,7 +167,12 @@ public class RoutingContextImpl implements RoutingContext {
 
       this.internalOnly = null;
 
-      mirrorOption = MirrorOption.enabled;
+      // once we set to disabled, we keep it always disabled.
+      // This is because the routing object used to route commands will 
disable this
+      // and it should stay that way no matter what
+      if (mirrorOption != MirrorOption.disabled) {
+         mirrorOption = MirrorOption.enabled;
+      }
 
       return this;
    }