-
Notifications
You must be signed in to change notification settings - Fork 317
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
On-deman backend heap and thread dump #8320
On-deman backend heap and thread dump #8320
Conversation
lib/scala/profiling-utils/src/main/java/org/enso/profiling/snapshot/HeapDumpGenerator.java
Outdated
Show resolved
Hide resolved
@@ -14,4 +14,10 @@ object ProfilingProtocol { | |||
/** A response to request to stop the profiling. */ | |||
case object ProfilingStopResponse | |||
|
|||
/** A request to create a memory snapshot. */ | |||
case object ProfilingSnapshotRequest |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Is it really profiling? Isn't it memory dump or "telemetry snapshot"?
lib/scala/profiling-utils/src/main/java/org/enso/profiling/snapshot/HeapDumpGenerator.java
Outdated
Show resolved
Hide resolved
@Test | ||
public void generateHeapDump() throws IOException { | ||
String heapDumpFileName = | ||
getClass().getSimpleName() + "-" + System.currentTimeMillis() + ".hprof"; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Might be easier to locate if this has a human-readable time format.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
It is supposed to be removed after the test anyway. The actual heap dump file created by the language server will be in a human-readable format.
Pull Request Description
close #8249
Changelog:
profiling/snapshot
request that takes a heap dump of the language server and puts it in theENSO_DATA_DIRECTORY/profiling
direcotryImportant Notes
Checklist
Please ensure that the following checklist has been satisfied before submitting the PR:
Scala,
Java,
and
Rust
style guides. In case you are using a language not listed above, follow the Rust style guide.
./run ide build
.