nicoloboschi opened a new pull request #12420:
URL: https://github.com/apache/pulsar/pull/12420
### Motivation
We often see on our CI the test `SimpleProducerConsumerTest` failing due to:
```
java.lang.AssertionError: expected [30] but found [29]
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.SimpleProducerConsumerTest.testMultiTopicsConsumerImplPauseForManualSubscription(SimpleProducerConsumerTest.java:3304)
```
### Modifications
- add a system property to multiply read timeouts for consumer
note: I'm aware this will not solve the whole problem, but I see them
failing only on CI where the system is likely slower, and we can chase them by
chances.
### Verifying this change
This change is a trivial rework / code cleanup without any test coverage.
### Documentation
Need to update docs?
- [x] `no-need-doc`
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
