You are viewing a plain text version of this content. The canonical link for it is here.
Posted to notifications@skywalking.apache.org by "Superskyyy (via GitHub)" <gi...@apache.org> on 2023/04/11 15:42:01 UTC

[GitHub] [skywalking] Superskyyy commented on issue #10672: [Bug] Python agent critical performance regression

Superskyyy commented on issue #10672:
URL: https://github.com/apache/skywalking/issues/10672#issuecomment-1503638043

   Right, using `event.wait(0)` is very close to busy waiting to me, since it constantly checks for if the event is set. It will cause high cpu usage.  Though I also don't know what happen previously and why it didn't cause similar performance degradation, will need a deeper look into the previous code.
   
   But meanwhile I think its good to add a wait(0.01 or even a bit more), the default 0 is definitely not friendly for production envs.


-- 
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: notifications-unsubscribe@skywalking.apache.org

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