RFR: 8273541: Cleaner Thread creates with normal priority instead of MAX_PRIORITY - 2
shade at openjdk.java.net
Fri Sep 10 06:25:23 UTC 2021
On Thu, 9 Sep 2021 10:14:48 GMT, kabutz <github.com+332398+kabutz at openjdk.org> wrote:
> …of MAX_PRIORITY-2 during refactoring
> Appears in Java 17 for the first time.
> During refactoring, the priority was changed from Thread.MAX_PRIORITY - 2 to instead state Thread.MIN_PRIORITY - 2, which results in a negative priority, and is thus set to Thread.NORM_PRIORITY. Thus the Cleaner by default now has threads with priority 5, instead of 8.
> The change was done in git revision # 992b50087d2ec8878dfcbbd1820a00b6b6bdf644 and label 8261036 by Claes Redestad
This looks good. @cl4es might want to confirm that change was not intentional.
Please change the PR title to "8273541: Cleaner Thread creates with normal priority instead of MAX_PRIORITY - 2" to get it automatically hooked by bots.
src/java.base/share/classes/jdk/internal/ref/CleanerImpl.java line 228:
> 226: super(cleaner, cleaner);
> 227: }
> 228: java
Changes requested by shade (Reviewer).
More information about the core-libs-dev