Improving Code Reviewer Recommendation: Accuracy, Latency, Workload, and Bystanders
Fuente:
arXiv
Saved in:
| Main Authors: | , , , , , , , |
|---|---|
| Format: | Preprint |
| Published: |
2023
|
| Subjects: | |
| Online Access: | |
| Tags: |
Add Tag
No Tags, Be the first to tag this record!
|
| _version_ | 1866913817514999808 |
|---|---|
| author | Rigby, Peter C. Rogers, Seth Saleem, Sadruddin Suresh, Parth Suskin, Daniel Riggs, Patrick Maddila, Chandra Nagappan, Nachiappan |
| author_facet | Rigby, Peter C. Rogers, Seth Saleem, Sadruddin Suresh, Parth Suskin, Daniel Riggs, Patrick Maddila, Chandra Nagappan, Nachiappan |
| contents | The code review team at Meta is continuously improving the code review process. To evaluate the new recommenders, we conduct three A/B tests which are a type of randomized controlled experimental trial.
Expt 1. We developed a new recommender based on features that had been successfully used in the literature and that could be calculated with low latency. In an A/B test on 82k diffs in Spring of 2022, we found that the new recommender was more accurate and had lower latency.
Expt 2. Reviewer workload is not evenly distributed, our goal was to reduce the workload of top reviewers. We then ran an A/B test on 28k diff authors in Winter 2023 on a workload balanced recommender. Our A/B test led to mixed results.
Expt 3. We suspected the bystander effect might be slowing down reviews of diffs where only a team was assigned. We conducted an A/B test on 12.5k authors in Spring 2023 and found a large decrease in the amount of time it took for diffs to be reviewed when a recommended individual was explicitly assigned.
Our findings also suggest there can be a discrepancy between historical back-testing and A/B test experimental findings. |
| format | Preprint |
| id |
arxiv_https___arxiv_org_abs_2312_17169 |
| institution | arXiv |
| publishDate | 2023 |
| record_format | arxiv |
| spellingShingle | Improving Code Reviewer Recommendation: Accuracy, Latency, Workload, and Bystanders Rigby, Peter C. Rogers, Seth Saleem, Sadruddin Suresh, Parth Suskin, Daniel Riggs, Patrick Maddila, Chandra Nagappan, Nachiappan Software Engineering The code review team at Meta is continuously improving the code review process. To evaluate the new recommenders, we conduct three A/B tests which are a type of randomized controlled experimental trial. Expt 1. We developed a new recommender based on features that had been successfully used in the literature and that could be calculated with low latency. In an A/B test on 82k diffs in Spring of 2022, we found that the new recommender was more accurate and had lower latency. Expt 2. Reviewer workload is not evenly distributed, our goal was to reduce the workload of top reviewers. We then ran an A/B test on 28k diff authors in Winter 2023 on a workload balanced recommender. Our A/B test led to mixed results. Expt 3. We suspected the bystander effect might be slowing down reviews of diffs where only a team was assigned. We conducted an A/B test on 12.5k authors in Spring 2023 and found a large decrease in the amount of time it took for diffs to be reviewed when a recommended individual was explicitly assigned. Our findings also suggest there can be a discrepancy between historical back-testing and A/B test experimental findings. |
| title | Improving Code Reviewer Recommendation: Accuracy, Latency, Workload, and Bystanders |
| topic | Software Engineering |
| url | https://arxiv.org/abs/2312.17169 |