Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

DeadLockDetectorPerformanceCounter does not send thread stack in TelemetryTrace #1263

Closed
jzanon opened this issue Jun 24, 2020 · 0 comments · Fixed by #1842
Closed

DeadLockDetectorPerformanceCounter does not send thread stack in TelemetryTrace #1263

jzanon opened this issue Jun 24, 2020 · 0 comments · Fixed by #1842

Comments

@jzanon
Copy link

jzanon commented Jun 24, 2020

Expected behavior

DeadLockDetectorPerformanceCounter shall send some thread stack elements within message of TraceTelemetry.

Actual behavior

DeadLockDetectorPerformanceCounter only send ThreadInfo without thread stack trace in TraceTelemetry.

To Reproduce

  • DeadLockDetectorPerformanceCounter enabled (default behavior)
  • When a deadlock occurs or is created for test purpose

Sample Application

None

System information

Please provide the following information:

  • SDK Version: applicationinsights-core-2.5.1.jar
  • OS type and version: All OS
  • Using spring-boot? No
  • Additional relevant libraries (with version, if applicable): No

Logs

TelemetryTrace message from Application Insights:
Suspected deadlocked threads: XXX-Startup-tasks Id=171 is in WAITING on lock=java.util.concurrent.locks.ReentrantLock$NonfairSync@48ca6b79 is owned by EE-ManagedThreadFactory-default-Thread-2 Id=170 | EE-ManagedThreadFactory-default-Thread-2 Id=170 is in BLOCKED on lock=org.jboss.weld.Container@416c2a69 is owned by XXX-Startup-tasks Id=171 |

Fix idea

Use getThreadInfo(threadId, MAX_STACK_TRACE) here


to have at least MAX_STACK_TRACE elements.

Nice-to-Have: Make MAX_STACK_TRACE configurable using ApplicationInsights.xml

trask pushed a commit that referenced this issue Oct 17, 2020
@ghost ghost added the Status: Fixed label Aug 12, 2021
@ghost ghost locked as resolved and limited conversation to collaborators Sep 12, 2021
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants