lhotari commented on code in PR #16011:
URL: https://github.com/apache/pulsar/pull/16011#discussion_r896436565
##########
tests/integration/src/test/java/org/apache/pulsar/tests/integration/functions/PulsarFunctionsTest.java:
##########
@@ -1584,7 +1584,7 @@ private void publishAndConsumeMessages(String inputTopic,
}
for (int i = 0; i < numMessages; i++) {
- Message<byte[]> msg = consumer.receive(30, TimeUnit.SECONDS);
+ Message<byte[]> msg = consumer.receive();
Review Comment:
> To me, 2 vCPUs might not be enough for the e2e tests, especially when
running all pulsar components with dockers. I could be wrong here.
2 vCPUs and 7GB RAM is plenty of computation power & RAM for the tests that
we have.
> If this timeout issue started happening only recently, then I agree that
we have a bug here. Please let me know if we do not want this change. The
intention is to make the test more stable for other PRs.
I am not convinced that we should remove the timeout. The problem must be
investigated, and the root cause should be fixed. There must be a bug in
production code if 30 seconds isn't sufficient.
--
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]
