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 2022/02/15 14:01:32 UTC

[GitHub] [pulsar] nicoloboschi opened a new pull request #14302: [tests] PulsarByteBufAllocator constructor mock can make parallel tests fail

nicoloboschi opened a new pull request #14302:
URL: https://github.com/apache/pulsar/pull/14302


   ### Motivation
   In the CI we have the option `reuseFork` = true. This means the process are reused across tests. 
   There are two tests that mock PulsarByteBufAllocator#DEFAULT field and this can lead the following tests with unexpected behaviours.
   
   The tests are `PulsarByteBufAllocatorDefaultTest` and `PulsarByteBufAllocatorOomThrowExceptionTest`
   
   Example of failure: https://github.com/apache/pulsar/runs/5200174968?check_suite_focus=true
   
   ### Modifications
   
   * Clean the mocked instance of PulsarByteBufAllocator after the tests
   
     
   - [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: commits-unsubscribe@pulsar.apache.org

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

[GitHub] [pulsar] merlimat merged pull request #14302: [tests] PulsarByteBufAllocator constructor mock can make parallel tests fail

Posted by GitBox <gi...@apache.org>.
merlimat merged pull request #14302:
URL: https://github.com/apache/pulsar/pull/14302


   


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

[GitHub] [pulsar] merlimat merged pull request #14302: [tests] PulsarByteBufAllocator constructor mock can make parallel tests fail

Posted by GitBox <gi...@apache.org>.
merlimat merged pull request #14302:
URL: https://github.com/apache/pulsar/pull/14302


   


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