Public Code Review

Pre-requisite

Before asking for code review, make sure your fix is applied on the most recent revision of the code you intend to work on, that there are no build errors on the supported platforms for the release you're working on, and that any included or otherwise applicable tests pass without failures on all relevant platforms. If you're not able to build or test on all supported platforms, you MUST let the reviewers know which ones you were able to build and test on, as well as whether you anticipate any issues on the platforms you haven't been able to build and test on.

Rule 0

Code reviews for public JDK 7 Update forests MUST be done publicly: Either through e-mail on jdk7u-dev@openjdk.java.net mailing list, or using some other suitable public mechanism. If a code review is not done on the jdk7u-dev@openjdk.java.net mailing list, as part of the approval request for inclusion of the fix into a public JDK 7 Updates forest, which you MUST send to the jdk7u-dev@openjdk.java.net mailing list, a URL for the public code review MUST be provided.

Rule 1

If the content of a changeset submitted for review for a public JDK 7 Update forest is the same patch as the corresponding changeset submitted or committed for JDK 8, then it does not have to be resubmitted. In that case the URL for the changeset in the JDK 8 repository will suffice.

Rule 2

If the content of a changeset submitted for review for a public JDK 7 Update forest differs from the corresponding JDK 8 changeset, or if there is no corresponding JDK 8 changeset, then the changeset MUST be submitted publicly for review:

• A very small changeset MAY be submitted as a simple patch, as described in http://openjdk.java.net/contribute/
• Other than that, changesets SHOULD be submitted as webrevs on the code review server, cr.openjdk.java.net.
• Alternatively, a URL for the webrev of the changeset MAY be provided

Rule 3

A changeset submitted for the JDK 7 Update mainline forest (jdk7u) MUST be reviewed by at least one reviewer. The maintainer responsible for approval of the changeset MAY request additional, specific reviewers to review the changeset, e. g. component leads.

Rule 4

A changeset submitted for a JDK 7 Update release forest (like jdk7u2, once it's branched off) MUST be reviewed by at least two reviewers. The maintainer responsible for approval of the changeset MAY request additional, specific reviewers to review the changeset, e. g. component leads.

Rule 5

When backporting a changeset from OpenJDK 8 to OpenJDK 7u, you SHOULD ensure you have agreement from the original author and reviewers that it's suitable for backporting. You SHOULD also cc the original author and reviewers on the jdk7u-dev@openjdk.java.net mailing list when requesting approval.