Hi, (I put this in the PR, but maybe drafts don’t get updates put on the mailing list?)
HeapDumpOverwrite sounds quite general, there are different ways of heap dumping and this doesn't affect all of them. This could be HeapDumpOnOutOfMemoryErrorOverwrite, which is a long option even for us 8-) so could be abbreviated perhaps, BUT that option might not be the way to do it… Is it really that all heap dumps should be permitted to an existing file, IF that file is a FIFO? From the email I think that's the problem? ..and if so, we have: 8267666: Add option to jcmd GC.heap_dump to use existing file https://bugs.openjdk.org/browse/JDK-8267666 So if jcmd to create a dump can use an existing file, -XX:OnError= can trigger such a jcmd? i.e. Maybe there is a way of doing what you want. A code change could still be required to make this easier for heap dumps on out of memory. Writing a JBS bug first is useful to define the problem. Thanks Kevin From: jdk-dev <jdk-dev-r...@openjdk.org> On Behalf Of Elon Azoulay Sent: 25 May 2023 21:21 To: Daniel Daugherty <daniel.daughe...@oracle.com> Cc: serviceability-dev@openjdk.org; jdk-...@openjdk.org Subject: Re: [External] : Re: Proposal: Add overwrite heapdump flag to java Sounds good, thanks so much! On Thu, May 25, 2023 at 12:13 PM <daniel.daughe...@oracle.com<mailto:daniel.daughe...@oracle.com>> wrote: Pointing the older thread to this newer incarnation would be a good idea. Dan On 5/25/23 3:11 PM, Elon Azoulay wrote: Hi Dan, That's great! Should I follow up in that thread as well? On Thu, May 25, 2023 at 12:04 PM <daniel.daughe...@oracle.com<mailto:daniel.daughe...@oracle.com>> wrote: Greetings, A similar request came up on the hotspot-dev alias back in 2022.11: https://mail.openjdk.org/pipermail/hotspot-dev/2022-November/066956.html Dan On 5/25/23 1:48 AM, David Holmes wrote: > Hi Elon, > > I would suggest taking this up on serviceability-dev - cc'd. > > On 25/05/2023 3:46 am, Elon Azoulay wrote: >> Hi, >> I submitted a pull request >> <https://github.com/openjdk/jdk/pull/13276<https://urldefense.com/v3/__https:/github.com/openjdk/jdk/pull/13276__;!!ACWV5N9M2RV99hQ!P4XNOUSVFXTWEIX8H89o1hhVY4MkuYC-SezupmhJPMkq4ytSpJqLz-jQjNkp906DjKBK_EuRIoL1MtBM2AGEbVqkI0M$>> >> to expose the overwrite >> flag to HeapDumpOnOutOfMemoryError. >> The flag is already exposed to jcmd as -overwrite.This is to >> facilitate creating a heapdump within a container native environment. >> We use this internally to dump the heap dump to a fifo so that we >> could read it in a separate container.Let me know what needs to be >> done in terms of creating an issue and getting my pr merged. > > Have you gone through the guide: > > https://openjdk.org/guide/ > > ? > > You will need a JBS issue and also a CSR request as this proposes to > add a new manageable product flag. > > Cheers, > David > >> I am a new contributor and would be happy to contribute more! >> >> Cheers, >> >> Elon
