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

Stall detection duplicate #1542

Merged
merged 9 commits into from
Oct 10, 2023
Merged

Stall detection duplicate #1542

merged 9 commits into from
Oct 10, 2023

Conversation

jtduffy
Copy link
Contributor

@jtduffy jtduffy commented Oct 6, 2023

Duplicate of #1536
Resolves #1527

Some notes:

  • Adds a feature which detects slow transactions and reports as SlowTransaction analytic events
  • Implementation hooks into transaction listener to maintain map of open transactions. Before harvest, it scans the map of open transactions and identifies the slowest that has been open longer than a threshold. It extracts metadata about that transaction, notably including the stacktrace of the thread that initialized it, and emits an event.
  • The implementation has a mechanism preventing the same transaction to be reported multiple times.
  • Disabled by default
  • Threshold is configurable, and defaults to 1s (i.e. any transaction open for longer than 1s is a candidate to be reported).

@jtduffy jtduffy mentioned this pull request Oct 6, 2023
@jtduffy jtduffy merged commit 2ad9c92 into main Oct 10, 2023
103 checks passed
@jtduffy jtduffy deleted the stall-detection-duplicate branch October 10, 2023 15:42
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Archived in project
Development

Successfully merging this pull request may close these issues.

Add stall detection to the NR java agent
3 participants