git.net

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

[Proposal] Code Reviewer List


Hi Flink Community,

I am working with Flink for last couple of months and still learning it. Recently, I have started as a contributor in Flink to perform some minor changes and I am very excited to start working on bigger tasks. 
Questions I always have in mind, when I make a PR, are “Who should I tag in my PR for code review?” and “How long does it take to get a first response from the reviewer?” 

Usually if there is a Jira ticket reported on the task, I will tag reporter as well as people who commented on it. Also, I look at other PRs with similar topics and try to find out who is doing code review for those PRs.

Here I would like to share a proposal to create a "Code Reviewer List" as part of Flink contribution guideline. This list would provide name of code reviewers, their availabilities, topics of interest as well as turnaround time for initial code review. This will provide good starting point for new contributors like myself to find a code reviewer based on a topic he/she is contributing. 

I would appreciate if Flink community shares thoughts on this proposal!

Cheers,
Yazdan