On Thu, 9 Sep 2021 10:14:48 GMT, kabutz <github.com+332398+kab...@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 Stray change. ------------- Changes requested by shade (Reviewer). PR: https://git.openjdk.java.net/jdk/pull/5439
