Mailing-List: contact dev-help@reef.apache.org; run by ezmlm
Precedence: bulk
Reply-To: dev@reef.apache.org
Subject: Re: code review
To: dev@reef.apache.org
References: 
 <DM2PR03MB319128A7BB1C8504D91346B90D60@DM2PR03MB319.namprd03.prod.outlook.com>
 <CADq0cj5_CK=8GqnJd7j_2hZu5+TUiLbjarcJJvQFR6_P1GosnQ@mail.gmail.com>
 <CAOq_m-O+AbaJLvcQtvcPm4-Ux8LUc=VndefH6Xyemm5LabBqVw@mail.gmail.com>
From: Markus Weimer <markus@weimo.de>
Message-ID: <56BA37DB.6080808@weimo.de>
Date: Tue, 9 Feb 2016 11:02:51 -0800
User-Agent: Mozilla/5.0 (Windows NT 10.0; WOW64; rv:38.0) Gecko/20100101
 Thunderbird/38.5.1
MIME-Version: 1.0
In-Reply-To: 
 <CAOq_m-O+AbaJLvcQtvcPm4-Ux8LUc=VndefH6Xyemm5LabBqVw@mail.gmail.com>
Content-Type: text/plain; charset=utf-8
Content-Transfer-Encoding: 7bit

+1 on asking people to help with the reviews. I believe we do this today
already, right?

Now, there is a line we shouldn't cross, and that is the idea of having
"maintainers" or "owners" of different parts of the code base. The code
is in whole owned by the community, and every committer should feel free
to commit to all parts of the code base. In fact, it is beneficial for
community health to review outside of our usual area of expertise, to
make sure that the code as a whole is most broadly understood.

Makes sense?

Markus


On 2016-02-09 08:17, Yunseong Lee wrote:
> +1 :-)
> 
> Thanks for the great suggestion, Julia!
> 
> Regards,
> Yunseong
> 
> On Tue, Feb 9, 2016 at 3:57 PM, Byung-Gon Chun <bgchun@gmail.com> wrote:
> 
>> Sounds good to me!
>> -Gon
>>
>> On Tue, Feb 9, 2016 at 3:22 PM, Julia Wang (QIUHE) <
>> Qiuhe.Wang@microsoft.com
>>> wrote:
>>
>>> When we use CodeFlow or some other tools, we used to specify required and
>>> optional code reviewers. This is to make sure the modified code are
>>> reviewed by people who also worked on that code before, or who are also
>>> working on the related portions. When we send a PR for review though
>>> GitHub, there is no place for us to specify who are the required
>> reviewers.
>>> I would suggest us to use @alias to specify required reviewers when
>>> seeding a PR for review. In this way, people won't miss any code review.
>>> For those who are not mentioned, they can always be optional reviewers.
>>>
>>> What do you think?
>>>
>>> Thanks,
>>> Julia
>>>
>>
>>
>>
>> --
>> Byung-Gon Chun
>>
>