Inverse the condition to clarify the logic

apache · Apr 2, 2021 · 2cb384b · 2cb384b
1 parent e4ac3a3
commit 2cb384b
Showing 1 changed file with 2 additions and 2 deletions.
diff --git a/...roker/src/main/java/org/apache/pulsar/broker/service/persistent/PersistentReplicator.java b/...roker/src/main/java/org/apache/pulsar/broker/service/persistent/PersistentReplicator.java
@@ -760,8 +760,8 @@ private void checkReplicatedSubscriptionMarker(Position position, MessageImpl<?>
 
         int markerType = msg.getMessageBuilder().getMarkerType();
 
-        if (!msg.getMessageBuilder().hasReplicatedFrom() ||
-                !remoteCluster.equals(msg.getMessageBuilder().getReplicatedFrom())) {
+        if (!(msg.getMessageBuilder().hasReplicatedFrom()
+                && remoteCluster.equals(msg.getMessageBuilder().getReplicatedFrom()))) {
             // Only consider markers that are coming from the same cluster that this
             // replicator instance is assigned to.
             // All the replicators will see all the markers, but we need to only process