[prev in list] [next in list] [prev in thread] [next in thread] 

List:       openjdk-serviceability-dev
Subject:    Re: Proposal: Add overwrite heapdump flag to java
From:       Elon Azoulay <elon.azoulay () gmail ! com>
Date:       2023-05-25 19:11:00
Message-ID: CAF6QBYZKTKvUfSRsFf=bPPVd8O7ToY3x+mQHOtaCaz8D_-exVw () mail ! gmail ! com
[Download RAW message or body]

Hi Dan,
That's great! Should I follow up in that thread as well?


On Thu, May 25, 2023 at 12:04 PM <daniel.daugherty@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> 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
>
>

[Attachment #3 (text/html)]

<div dir="ltr">Hi Dan,<div>That&#39;s great! Should I follow up in that thread as \
well?</div><div><br></div></div><br><div class="gmail_quote"><div dir="ltr" \
class="gmail_attr">On Thu, May 25, 2023 at 12:04 PM &lt;<a \
href="mailto:daniel.daugherty@oracle.com">daniel.daugherty@oracle.com</a>&gt; \
wrote:<br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px \
0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">Greetings,<br> <br>
A similar request came up on the hotspot-dev alias back in 2022.11:<br>
<br>
<a href="https://mail.openjdk.org/pipermail/hotspot-dev/2022-November/066956.html" \
rel="noreferrer" target="_blank">https://mail.openjdk.org/pipermail/hotspot-dev/2022-November/066956.html</a><br>
 <br>
Dan<br>
<br>
<br>
On 5/25/23 1:48 AM, David Holmes wrote:<br>
&gt; Hi Elon,<br>
&gt;<br>
&gt; I would suggest taking this up on serviceability-dev - cc&#39;d.<br>
&gt;<br>
&gt; On 25/05/2023 3:46 am, Elon Azoulay wrote:<br>
&gt;&gt; Hi,<br>
&gt;&gt; I submitted a pull request <br>
&gt;&gt; &lt;<a href="https://github.com/openjdk/jdk/pull/13276" rel="noreferrer" \
target="_blank">https://github.com/openjdk/jdk/pull/13276</a>&gt; to expose the \
overwrite <br> &gt;&gt; flag to HeapDumpOnOutOfMemoryError.<br>
&gt;&gt; The flag is already exposed to jcmd as -overwrite.This is to <br>
&gt;&gt; facilitate creating a heapdump within a container native environment.<br>
&gt;&gt; We use this internally to dump the heap dump to a fifo so that we <br>
&gt;&gt; could read it in a separate container.Let me know what needs to be <br>
&gt;&gt; done in terms of creating an issue and getting my pr merged.<br>
&gt;<br>
&gt; Have you gone through the guide:<br>
&gt;<br>
&gt; <a href="https://openjdk.org/guide/" rel="noreferrer" \
target="_blank">https://openjdk.org/guide/</a><br> &gt;<br>
&gt; ?<br>
&gt;<br>
&gt; You will need a JBS issue and also a CSR request as this proposes to <br>
&gt; add a new manageable product flag.<br>
&gt;<br>
&gt; Cheers,<br>
&gt; David<br>
&gt;<br>
&gt;&gt; I am a new contributor  and would be happy to contribute more!<br>
&gt;&gt;<br>
&gt;&gt; Cheers,<br>
&gt;&gt;<br>
&gt;&gt; Elon<br>
<br>
</blockquote></div>



[prev in list] [next in list] [prev in thread] [next in thread] 

Configure | About | News | Add a list | Sponsored by KoreLogic