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 2021/08/24 11:37:15 UTC

[GitHub] [pulsar] BewareMyPower commented on issue #11760: Flaky-test: C++ test BasicEndToEndTest.testLookupThrottling

BewareMyPower commented on issue #11760:
URL: https://github.com/apache/pulsar/issues/11760#issuecomment-904561514


   Current not. From the existing logs, it's because C++ client encountered segmentation fault during client's `shutdown()`, more accurately, the `ExecutorProvider`'s close phase (but there's no log).
   
   https://github.com/apache/pulsar/blob/ac5fce55e1394698072d0cbdd0e289bd289a217e/pulsar-client-cpp/lib/ClientImpl.cc#L571-L573
   
   https://github.com/apache/pulsar/blob/ac5fce55e1394698072d0cbdd0e289bd289a217e/pulsar-client-cpp/lib/ExecutorService.cc#L98-L103
   
   The only clue that I found is that `ExecutorServiceProvider::close` is not thread safe because `executors_` is protected by `mutex_` in `ExecutorServiceProvider::get` while `mutex_` is not locked in `close` method.
   
   I'll fix the theoretical thread-safe problem soon, but not guarantee it can fix the flaky test.


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