RFR: 8335108: Build error after JDK-8333658 due to class templates

Kim Barrett kbarrett at openjdk.org
Thu Jun 27 05:57:13 UTC 2024


On Thu, 27 Jun 2024 05:26:12 GMT, Julian Waters <jwaters at openjdk.org> wrote:

> I guess you learn something new every day, since this is the first time I'm hearing about the 24 hour rule (I always thought the only HotSpot requirement was 2 Reviewers, besides needing a 2 week waiting periods for Style Guide changes, never knew you had to wait 24 hours for regular ones too). It's a good thing this change was trivial in the end

It's a JDK rule, not specific to HotSpot.  
https://openjdk.org/guide/
Life of a PR
6. Allow enough time for review

https://github.com/openjdk/guide/blame/95f1cd24c657c7837c359c7ba1a80e15319bcd15/src/guide/working-with-pull-requests.md#L77-L80

I don't remember where that requirement originally came from, before being incorporated into the guide.  I thought
there was someplace old but public that prescribed 1 Reviewer and 24 hours, but can't find it right now.  Maybe it was in the openjdk.org contributing or sponsoring pages and got cleaned up when added to the guide?

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

PR Comment: https://git.openjdk.org/jdk/pull/19890#issuecomment-2193860942


More information about the hotspot-dev mailing list