ascrutae commented on issue #2827: Improve ContextManager class of sniffer 
module
URL: https://github.com/apache/skywalking/pull/2827#issuecomment-498995767
 
 
   Many scenarios may cause the issue. I’m not clear about your situation, The 
situation that the memory leak is that the trace isn't finished, and the 
`StackDepth` don’t remove from thread local. And the issue maybe caused by two 
thread had to share the StackDepth variable.
   
   
   Original Message
   Sender:小水怪[email protected]
   Recipient:apache/[email protected]
   Cc:Zhang" [email protected]; [email protected]
   Date:Wednesday, Jun 5, 2019 16:27
   Subject:Re: [apache/skywalking] Improve ContextManager class of 
sniffermodule (#2827)
   
   
   Thanks for your advice @ascrutae 。But , CONTEXT has been removed, meaning 
that TraceContext has been removed from ThreadLocal, so I think that there is 
no need to keep RUNTIME-CONTEXT. My thoughts are consistent with you. @wu-sheng
   —
   You are receiving this because you were mentioned.
   Reply to this email directly, view it on GitHub, or mute the thread.

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


With regards,
Apache Git Services

Reply via email to