RFR: 8317755: G1: Periodic GC interval should test for the last whole heap GC [v3]

[Thomas Schatzl]

On Mon, 27 Nov 2023 20:47:30 GMT, Y. Srinivas Ramakrishna <ysr at openjdk.org> wrote:

> LGTM. Very useful when there is no direct feedback loop from resource pressure that we might otherwise have used for signaling the need for a whole heap GC, so is a good escape hatch to put in the hands of service owners. Of course, like any escape hatch, it could be misused, but I don't think that is a concern about ejection seats from fighter jets for example. We trust those who fly the jets not to eject for no reason.
> 
> I would suggest formally dissociating this from the JEP itself. The fix ups to the JEP's and its implementation shortcomings can be addressed fully in the fullness of time. My USD 0.02.

I do not think breaking existing functionality to add new functionality given that the former is widely publicized via the JEP for the default collector and "fixing it up later" is the proper way forward.
The only advantage I can see with this approach is that this is a simple change, but implementing the new functionality properly is likely not that much more work anyway.

-------------

PR Comment: https://git.openjdk.org/jdk/pull/16107#issuecomment-1835762014