lukecwik commented on a change in pull request #17151:
URL: https://github.com/apache/beam/pull/17151#discussion_r832452714
##########
File path:
runners/google-cloud-dataflow-java/src/main/java/org/apache/beam/runners/dataflow/options/DataflowPipelineDebugOptions.java
##########
@@ -216,6 +216,25 @@ public Dataflow create(PipelineOptions options) {
void setDumpHeapOnOOM(boolean dumpHeapBeforeExit);
+ /**
+ * If true, save a JFR profile when GC thrashing is first detected. The
profile will run for the
+ * amount of time set by --jfrRecordingDurationSec, or 60 seconds by default.
+ *
+ * <p>Note, JFR profiles are only supported on java 9 and up.
+ */
+ @Description(
+ "If true, save a JFR profile before killing a thread or process "
+ + "which is GC thrashing or out of memory. Only available on java 9
or up")
+ boolean getRecordJfrOnGcThrashing();
Review comment:
This feature is nice, can we add it to sdks/java/harness as well so all
portable runners can get it instead of just Dataflow?
See
https://github.com/apache/beam/blob/master/sdks/java/core/src/main/java/org/apache/beam/sdk/options/SdkHarnessOptions.java
and
https://github.com/apache/beam/blob/master/sdks/java/harness/src/main/java/org/apache/beam/fn/harness/status/MemoryMonitor.java
--
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: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
