[fix][test] entry filter makes no sense (apache#17170)

gaozhangmin · gavingaozhangmin · web-flow · commit c4bd7ae15a1e · 2022-08-22T14:25:04.000+08:00
* [fix][test] entry filter makes no sense because trackDelayedDelivery always return true

* apply comments

Co-authored-by: gavingaozhangmin &lt;gavingaozhangmin@didiglobal.com&gt;
diff --git a/pulsar-broker/src/test/java/org/apache/pulsar/broker/service/AbstractBaseDispatcherTest.java b/pulsar-broker/src/test/java/org/apache/pulsar/broker/service/AbstractBaseDispatcherTest.java
@@ -214,8 +214,7 @@ protected boolean isConsumersExceededOnSubscription() {
 
         @Override
         public boolean trackDelayedDelivery(long ledgerId, long entryId, MessageMetadata msgMetadata) {
-            //for test.
-            return true;
+            return msgMetadata.hasDeliverAtTime();
         }
 
         @Override

Original file line number	Diff line number	Diff line change
`@@ -214,8 +214,7 @@ protected boolean isConsumersExceededOnSubscription() {`
`214`	`214`
`215`	`215`	`@Override`
`216`	`216`	`public boolean trackDelayedDelivery(long ledgerId, long entryId, MessageMetadata msgMetadata) {`
`217`		`- //for test.`
`218`		`- return true;`
	`217`	`+ return msgMetadata.hasDeliverAtTime();`
`219`	`218`	`}`
`220`	`219`
`221`	`220`	`@Override`