>=2 reviewers imply at least two have "reviewer" role?

Dalibor Topic dalibor.topic at oracle.com
Wed Jun 19 02:52:58 PDT 2013


On 6/19/13 11:29 AM, Alan Bateman wrote:
> On 19/06/2013 09:03, Seán Coffey wrote:
>> Leonid,
>>
>> Phase 2 requires two reviewers. Can you specify who has reviewed ?
> Nothing to do with this Leonid's request but does "two reviewers" imply that both have "reviewer" role (or is one with "reviewer" role okay)? It would be good to make this clear on the phase2 push approval page too.

They should both have the Reviewer role.

Would changing "At least two reviewers are required for phase 2." in http://openjdk.java.net/projects/jdk7u/phase2/phase2-approval-template.html 
to "At least two reviewers, from the set of Reviewers listed on the JDK 7 Update census page [0] are required for phase 2." make it clear? If so,
I'll formally propose it for discussion later today.

[0] http://openjdk.java.net/census#jdk7u
-- 
Oracle <http://www.oracle.com>
Dalibor Topic | Principal Product Manager
Phone: +494089091214 <tel:+494089091214> | Mobile: +491737185961 <tel:+491737185961>
Oracle Java Platform Group

ORACLE Deutschland B.V. & Co. KG | Kühnehöfe 5 | 22761 Hamburg

ORACLE Deutschland B.V. & Co. KG
Hauptverwaltung: Riesstr. 25, D-80992 München
Registergericht: Amtsgericht München, HRA 95603
Geschäftsführer: Jürgen Kunz

Komplementärin: ORACLE Deutschland Verwaltung B.V.
Hertogswetering 163/167, 3543 AS Utrecht, Niederlande
Handelsregister der Handelskammer Midden-Niederlande, Nr. 30143697
Geschäftsführer: Alexander van der Ven, Astrid Kepper, Val Maher

Green Oracle <http://www.oracle.com/commitment> Oracle is committed to developing practices and products that help protect the environment



More information about the jdk7u-dev mailing list