Flaky-test: V1_ProducerConsumerTest.testBlockUnackedConsumerRedeliverySpecificMessagesCloseConsumerWhileProduce

[codelipenghui]

example failure

  Error:  Tests run: 107, Failures: 25, Errors: 0, Skipped: 75, Time elapsed: 33.278 s <<< FAILURE! - in org.apache.pulsar.client.api.v1.V1_ProducerConsumerTest
  Error:  testBlockUnackedConsumerRedeliverySpecificMessagesCloseConsumerWhileProduce(org.apache.pulsar.client.api.v1.V1_ProducerConsumerTest)  Time elapsed: 1.687 s  <<< FAILURE!
  java.lang.AssertionError: expected [20] but found [10]
  	at org.testng.Assert.fail(Assert.java:99)
  	at org.testng.Assert.failNotEquals(Assert.java:1037)
  	at org.testng.Assert.assertEqualsImpl(Assert.java:140)
  	at org.testng.Assert.assertEquals(Assert.java:122)
  	at org.testng.Assert.assertEquals(Assert.java:907)
  	at org.testng.Assert.assertEquals(Assert.java:917)
  	at org.apache.pulsar.client.api.v1.V1_ProducerConsumerTest.testBlockUnackedConsumerRedeliverySpecificMessagesCloseConsumerWhileProduce(V1_ProducerConsumerTest.java:1703)
  	at java.base/jdk.internal.reflect.NativeMethodAccessorImpl.invoke0(Native Method)
  	at java.base/jdk.internal.reflect.NativeMethodAccessorImpl.invoke(NativeMethodAccessorImpl.java:77)
  	at java.base/jdk.internal.reflect.DelegatingMethodAccessorImpl.invoke(DelegatingMethodAccessorImpl.java:43)
  	at java.base/java.lang.reflect.Method.invoke(Method.java:568)
  	at org.testng.internal.MethodInvocationHelper.invokeMethod(MethodInvocationHelper.java:132)
  	at org.testng.internal.InvokeMethodRunnable.runOne(InvokeMethodRunnable.java:45)
  	at org.testng.internal.InvokeMethodRunnable.call(InvokeMethodRunnable.java:73)
  	at org.testng.internal.InvokeMethodRunnable.call(InvokeMethodRunnable.java:11)
  	at java.base/java.util.concurrent.FutureTask.run(FutureTask.java:264)
  	at java.base/java.util.concurrent.ThreadPoolExecutor.runWorker(ThreadPoolExecutor.java:1136)
  	at java.base/java.util.concurrent.ThreadPoolExecutor$Worker.run(ThreadPoolExecutor.java:635)
  	at java.base/java.lang.Thread.run(Thread.java:833)

[RobertIndie]

Hi, @codelipenghui I think this should be related to this PR: #16670.
The logic of the test itself should be wrong, but the fix by this PR causes the test to fail instead. I think we should fix this test by replacing this line

pulsar/pulsar-broker/src/test/java/org/apache/pulsar/client/api/v1/V1_ProducerConsumerTest.java

Line 1703 in 3d4fa00

assertEquals(messages1.size(), receiverQueueSize);

with

assertEquals(messages1.size(), unAckedMessagesBufferSize);