HotSpot Style Guide change process
David Holmes
david.holmes at oracle.com
Wed Dec 2 01:25:01 UTC 2020
Hi Kim,
"rough consensus" has good precedents and is something we should follow
here IMO. OpenJDK by-laws are for the formal parts of the OpenJDK
processes themselves, not for everything single thing done under the
OpenJDK banner.
Cheers,
David
On 1/12/2020 4:43 pm, Kim Barrett wrote:
> When the Style Guide was updated for C++11/14 usage and other cleanup
> (JDK-8247976), a change process was added to it. It says
>
> Proposed changes should be discussed on the
> [HotSpot Developers](mailto:hotspot-dev at openjdk.java.net) mailing
> list, and approved by
> [rough consensus](https://en.wikipedia.org/wiki/Rough_consensus) of
> the [HotSpot Group](https://openjdk.java.net/census#hotspot) Members.
> The Group Lead determines whether consensus has been reached.
> Changes are likely to be cautious and incremental, since HotSpot
> coders have been using these guidelines for years.
>
> Vladimir Kozlov has questioned this process and suggests another:
>
>> I looked again on voting description for Style Guide changes. And it
>> references to `rough consensus` which is not in OpenJDK bylaws :
>> https://github.com/openjdk/jdk/blob/master/doc/hotspot-style.html#L69
>>
>> I think it is bug (separate from these changes) and should be fixed by
>> using our rule http://openjdk.java.net/bylaws#three-vote-consensus
>> With Project Lead final vote we will need at least 2 other members votes
>> during 2 weeks review period. I think it is similar to `rough consensus`.
>
> Use of "rough consensus" with an arbiter has precedent within the OpenJDK,
> though it's not used in the bylaws. The current text quoted above was based
> on
>
> https://cr.openjdk.java.net/~mr/jep/jep-2.0-02.html
> JEP 2.0, draft 2 - see "Planning", 4th bullet
>
> and
>
> https://openjdk.java.net/groups/vulnerability/
> OpenJDK Vulnerability Group - see "Making decisions"
>
> So both the JEP Process and the Vulnerability Group use rough consensus,
> with the relevant lead as the arbiter. (The V-Team page links to an IETF
> document describing rough consensus. The HotSpot Style Guide links to
> the same Wikipedia article as the JEP 2.0 Process; that Wikipedia article
> references the IETF page.)
>
> I called out the new change process specifically in the initial RFR email
> for JDK-8247976, but that part didn't generate much (any?) discussion.
>
> If we don't like it we are certainly free to change. And if Vladimir
> doesn't like it then we should change, since as HotSpot Lead he's fairly
> central to that process. But I don't think there's a "lack of precedent"
> or "not appropriate for OpenJDK" argument for changing.
>
> I chose that process because I didn't want it to be too easy to make changes
> to the Style Guide. In particular, I think it should require more effort
> than is involved in a typical RFR.
>
> Even if we don't change, and especially if we do, some further refinement of
> the mechanics may be needed. I think the current mechanics may have a
> problem that the request shows up in one's mailbox looking pretty much like
> an ordinary RFR. When I was originally thinking about that text we were in a
> world of pure email change requests, and the document was still a wiki page.
> With the move of the document to the jdk repository and the move to github
> and PRs + Skara support for changes, the mechanics are different. It might
> be better if it could be subject tagged differently, as we do with Call for
> Vote emails. There are probably things we could do about that, if others
> agree that's a problem. Or maybe some of you see other problems?
>
More information about the hotspot-dev
mailing list