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 2019/10/22 16:22:25 UTC

[GitHub] [pulsar] merlimat opened a new pull request #5439: Multiple unit tests improvements

merlimat opened a new pull request #5439: Multiple unit tests improvements
URL: https://github.com/apache/pulsar/pull/5439
 
 
   ### Motivation
   
   Unit tests for `pulsar-broker` module have a very disappointing success rate, many flaky tests and a very long execution time.
   
   To improve situation, we need to:
    * Fix most egregious flaky tests
    * Reuse JVM forks when running tests. Currently we're spinning up a new JVM for *each* test class. That takes a huge amount of CPU time.
   * Run tests in parallel. Many tests are doing something and then waiting for something to happen. We can reduce execution by running in parallel
   
   ### Modifications
    
    * Fixed service port conflicts 
    * Avoid the ZK session watcher to take down the JVM when it fail to talk with ZK
    * Make sure we release all the resources at the end of the tests
   

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


With regards,
Apache Git Services