RFR: 8252141: Rename G1YoungRemSetSamplingThread to better reflect its purpose

Kim Barrett kim.barrett at oracle.com
Thu Aug 27 06:24:49 UTC 2020

> On Aug 26, 2020, at 3:18 AM, stefan.johansson at oracle.com wrote:
> Hi Kim,
> Thanks for reviewing.
> On 2020-08-25 04:44, Kim Barrett wrote:
>>> On Aug 24, 2020, at 5:36 PM, stefan.johansson at oracle.com wrote:
>>> Hi,
>>> Please review this rename of the G1 Young RemSet Sampling thread.
>>> Webrev: http://cr.openjdk.java.net/~sjohanss/8252141/00/
>>> JBS: https://bugs.openjdk.java.net/browse/JDK-8252141
>>> Summary
>>> The G1YoungRemSetSamplingThread does more than just remset sampling, it is also responsible for the scheduling of periodic GCs. Going forward even more taslks will likely be added to this thread and it should be renamed. The proposal in this patch is to rename it to G1ServiceThread.
>>> It also includes renaming some members and other related functions.
>>> Testing
>>> Mach5 tier1-3
>>> Thanks,
>>> Stefan
>> Generally looks good.
>> src/hotspot/share/gc/g1/g1ServiceThread.hpp
>>   30 // The G1ServiceThread is used to do a number of different task:
>> Pluralize "task"
>> Also add "periodically", as in "used to periodically do".
> Fixed.
>> I also noticed that G1ConcRefinementServiceIntervalMillis is kind of
>> misnamed and described, though with the change to "ServiceThread"
>> naming it's actually less poorly named than previously.
>> Changing the name is a bit of a pain, since it's a product option.
>> But the description should be fixed.
> Nice catch, updated in:
> Full: http://cr.openjdk.java.net/~sjohanss/8252141/01/
> Inc: http://cr.openjdk.java.net/~sjohanss/8252141/00-01/

Looks good.

> I also filed a issue to take care of the name at some point:
> https://bugs.openjdk.java.net/browse/JDK-8252358

Yes, that’s what I had in mind.  Thanks.

More information about the hotspot-gc-dev mailing list