You are viewing a plain text version of this content. The canonical link for it is here.

Posted to commits@pulsar.apache.org by GitBox <gi...@apache.org> on 2020/05/28 01:21:04 UTC

[GitHub] [pulsar] merlimat opened a new pull request #7071: C++ tests should use sensible timeouts for receive

merlimat opened a new pull request #7071:
URL: https://github.com/apache/pulsar/pull/7071


   ### Motivation
   
   A bunch of tests were using 100ms as the timeout for receive. 100ms is
   way too short. The broker for these tests is standalone, so everything
   is running in one process on one machine. I/O, locks, GC etc can
   easily make reads take longer than 100ms.
   
   We are testing functionality, not performance with these tests, so
   I've increased the timeout to 3s.
   


----------------------------------------------------------------
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.

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

[GitHub] [pulsar] sijie merged pull request #7071: C++ tests should use sensible timeouts for receive

Posted by GitBox <gi...@apache.org>.

sijie merged pull request #7071:
URL: https://github.com/apache/pulsar/pull/7071


   


----------------------------------------------------------------
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.

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