> Hi,
>
> Could I have a review of this patch?
>
> In the current implementation, HeapDumpBeforeFullGC/AfterFullGC will generate
> dumps for every FGC, increasing the risk of disk full.
>
> This patch introduces a new option 'FullGCHeapDumpLimit' to limit the number
> of dumps triggered by Hea
On Sat, 16 Dec 2023 04:36:57 GMT, Denghui Dong wrote:
>> Hi,
>>
>> Could I have a review of this patch?
>>
>> In the current implementation, HeapDumpBeforeFullGC/AfterFullGC will
>> generate dumps for every FGC, increasing the risk of disk full.
>>
>> This patch introduces a new option 'FullG
On Sat, 16 Dec 2023 04:36:57 GMT, Denghui Dong wrote:
>> Hi,
>>
>> Could I have a review of this patch?
>>
>> In the current implementation, HeapDumpBeforeFullGC/AfterFullGC will
>> generate dumps for every FGC, increasing the risk of disk full.
>>
>> This patch introduces a new option 'FullG
On Sat, 16 Dec 2023 04:36:57 GMT, Denghui Dong wrote:
>> Hi,
>>
>> Could I have a review of this patch?
>>
>> In the current implementation, HeapDumpBeforeFullGC/AfterFullGC will
>> generate dumps for every FGC, increasing the risk of disk full.
>>
>> This patch introduces a new option 'FullG
On Thu, 4 Jan 2024 03:19:17 GMT, Denghui Dong wrote:
>> Denghui Dong has updated the pull request incrementally with one additional
>> commit since the last revision:
>>
>> refine description
>
> Thanks for the review.
@D-D-H the new test is failing in our CI in tier3 on all platforms:
Err
On Sat, 16 Dec 2023 04:36:57 GMT, Denghui Dong wrote:
>> Hi,
>>
>> Could I have a review of this patch?
>>
>> In the current implementation, HeapDumpBeforeFullGC/AfterFullGC will
>> generate dumps for every FGC, increasing the risk of disk full.
>>
>> This patch introduces a new option 'FullG
On Thu, 4 Jan 2024 03:19:17 GMT, Denghui Dong wrote:
>> Denghui Dong has updated the pull request incrementally with one additional
>> commit since the last revision:
>>
>> refine description
>
> Thanks for the review.
> @D-D-H the new test is failing in our CI in tier3 on all platforms:
>