Re: [PR] [fix][test] Fix OneWayReplicatorTest.testProbBKErrorWhenReplicating timeout [pulsar]

["merlimat (via GitHub)" <gi...@apache.org>]

merlimat commented on PR #26581:
URL: https://github.com/apache/pulsar/pull/26581#issuecomment-6246216507

   Thanks @hanmz for digging into this. Since this PR was opened, #26620 changed both the replicator and this test. A failed read no longer stalls replication until the backoff timer fires, because producer acks resume reads. The test now expects the backlog to drain within 30 s instead of 600 s. So the 45 s method timeout matches the work again, and raising it to 300 s would only make a regression slower to show up. I'm closing this as superseded. If you still see `testProbBKErrorWhenReplicating` time out on current master, please open a `[Test] Flaky` issue with the CI log and we'll look at the drain path again.
   


-- 
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: commits-unsubscribe@pulsar.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org