flink-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From Fabian Hueske <fhue...@gmail.com>
Subject Re: [DISCUSS] (Not) tagging reviewers
Date Mon, 16 Jan 2017 11:45:23 GMT
Thanks for bringing this up Stephan.
I completely agree with you.

Cheers, Fabian

2017-01-16 12:42 GMT+01:00 Stephan Ewen <sewen@apache.org>:

> Hi!
>
> I have seen that recently many pull requests designate reviews by writing
> "@personA review please" or so.
>
> I am personally quite strongly against that, I think it hurts the community
> work:
>
>   - The same few people get usually "designated" and will typically get
> overloaded and often not do the review.
>
>   - At the same time, this discourages other community members from looking
> at the pull request, which is totally undesirable.
>
>   - In general, review participation should be "pull based" (person decides
> what they want to work on) not "push based" (random person pushes work to
> another person). Push-based just creates the wrong feeling in a community
> of volunteers.
>
>   - In many cases the designated reviews are not the ones most
> knowledgeable in the code, which is understandable, because how should
> contributors know whom to tag?
>
>
> Long story short, why don't we just drop that habit?
>
>
> Greetings,
> Stephan
>

Mime
  • Unnamed multipart/alternative (inline, None, 0 bytes)
View raw message