TakaHiR07 commented on issue #25081:
URL: https://github.com/apache/pulsar/issues/25081#issuecomment-3673807341

   > [@TakaHiR07](https://github.com/TakaHiR07) Do you have a chance to take a 
look? This test was added in 
[#24980](https://github.com/apache/pulsar/pull/24980)
   
   @lhotari 
   I can't find the root cause of this flaky test. I add some logs and have ran 
the test in personal ci three times, and all of them pass. You can see here, 
https://github.com/TakaHiR07/pulsar/pull/24 
   
   After check the code, I prefer the code logic is ok and it is more likely a 
problem in test. I suspect the following possibilities:
   1. The process is fine, but the logAppender do not output the log 
information.
   2. The logic inside readMorePoliciesAsync() is executed in another 
single-thread "broker-client-shared-internal-executor", and this logic runs 
slowly.
   3. There are two SystemTopicBasedTopicPoliciesService in test, original one 
and spy one. Both of them do  prepareInitPoliciesCacheAsync() and use the same 
thread "broker-client-shared-internal-executor". I’m not sure whether it make 
the process slowly or stuck the process. We can remove the original one in test 
code, but I wonder whether it is the root cause.


-- 
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: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]

Reply via email to