Flaky-test: PersistentStreamingDispatcherBlockConsumerTest. testBlockBrokerDispatching

PersistentStreamingDispatcherBlockConsumerTest is flaky. The testBlockBrokerDispatching test method fails sporadically.

example failure

  Error:  testBlockBrokerDispatching(org.apache.pulsar.broker.service.persistent.PersistentStreamingDispatcherBlockConsumerTest)  Time elapsed: 10.142 s  <<< FAILURE!
1148
  org.testng.internal.thread.ThreadTimeoutException: Method org.apache.pulsar.client.api.DispatcherBlockConsumerTest.testBlockBrokerDispatching() didn't finish within the time-out 10000

Full exception stacktrace

Error:  testBlockBrokerDispatching(org.apache.pulsar.broker.service.persistent.PersistentStreamingDispatcherBlockConsumerTest)  Time elapsed: 10.142 s  <<< FAILURE!
1148
  org.testng.internal.thread.ThreadTimeoutException: Method org.apache.pulsar.client.api.DispatcherBlockConsumerTest.testBlockBrokerDispatching() didn't finish within the time-out 10000
1149
  	at org.testng.internal.MethodInvocationHelper.invokeWithTimeoutWithNewExecutor(MethodInvocationHelper.java:371)
1150
  	at org.testng.internal.MethodInvocationHelper.invokeWithTimeout(MethodInvocationHelper.java:282)
1151
  	at org.testng.internal.TestInvoker.invokeMethod(TestInvoker.java:605)
1152
  	at org.testng.internal.TestInvoker.retryFailed(TestInvoker.java:214)
1153
  	at org.testng.internal.MethodRunner.runInSequence(MethodRunner.java:58)
1154
  	at org.testng.internal.TestInvoker$MethodInvocationAgent.invoke(TestInvoker.java:822)
1155
  	at org.testng.internal.TestInvoker.invokeTestMethods(TestInvoker.java:147)
1156
  	at org.testng.internal.TestMethodWorker.invokeTestMethods(TestMethodWorker.java:146)
1157
  	at org.testng.internal.TestMethodWorker.run(TestMethodWorker.java:128)
1158
  	at java.base/java.util.ArrayList.forEach(ArrayList.java:1541)
1159
  	at org.testng.TestRunner.privateRun(TestRunner.java:764)
1160
  	at org.testng.TestRunner.run(TestRunner.java:585)
1161
  	at org.testng.SuiteRunner.runTest(SuiteRunner.java:384)
1162
  	at org.testng.SuiteRunner.runSequentially(SuiteRunner.java:378)
1163
  	at org.testng.SuiteRunner.privateRun(SuiteRunner.java:337)
1164
  	at org.testng.SuiteRunner.run(SuiteRunner.java:286)
1165
  	at org.testng.SuiteRunnerWorker.runSuite(SuiteRunnerWorker.java:53)
1166
  	at org.testng.SuiteRunnerWorker.run(SuiteRunnerWorker.java:96)
1167
  	at org.testng.TestNG.runSuitesSequentially(TestNG.java:1218)
1168
  	at org.testng.TestNG.runSuitesLocally(TestNG.java:1140)
1169
  	at org.testng.TestNG.runSuites(TestNG.java:1069)
1170
  	at org.testng.TestNG.run(TestNG.java:1037)
1171
  	at org.apache.maven.surefire.testng.TestNGExecutor.run(TestNGExecutor.java:135)
1172
  	at org.apache.maven.surefire.testng.TestNGDirectoryTestSuite.executeSingleClass(TestNGDirectoryTestSuite.java:112)
1173
  	at org.apache.maven.surefire.testng.TestNGDirectoryTestSuite.executeLazy(TestNGDirectoryTestSuite.java:123)
1174
  	at org.apache.maven.surefire.testng.TestNGDirectoryTestSuite.execute(TestNGDirectoryTestSuite.java:90)
1175
  	at org.apache.maven.surefire.testng.TestNGProvider.invoke(TestNGProvider.java:146)
1176
  	at org.apache.maven.surefire.booter.ForkedBooter.invokeProviderInSameClassLoader(ForkedBooter.java:384)
1177
  	at org.apache.maven.surefire.booter.ForkedBooter.runSuitesInProcess(ForkedBooter.java:345)
1178
  	at org.apache.maven.surefire.booter.ForkedBooter.execute(ForkedBooter.java:126)
1179
  	at org.apache.maven.surefire.booter.ForkedBooter.main(ForkedBooter.java:418)

[Jason918]

Checked with the code, I think it’s most likely because of the receive timeout setting is too small in line 821, it should use receive()

@Jason918 good catch! Could it fix the various failures listed in my previous comment?

OK，I will look into these.

[lhotari]

Checked with the code, I think it’s most likely because of the receive timeout setting is too small in line 821, it should use receive()

@Jason918 good catch! Could it fix the various failures listed in my previous comment?