research-article

Removing stagnation from modern code review

Authors:

SPLASH Companion 2016: Companion Proceedings of the 2016 ACM SIGPLAN International Conference on Systems, Programming, Languages and Applications: Software for Humanity

Pages 43 - 44

https://doi.org/10.1145/2984043.2989224

Published: 20 October 2016 Publication History

Get Access

Abstract

Finding defects efficently is one of the major problems in software development, a problem that often still relies largely on human inspection of code to find defects. Many software development projects use code reviews as a mean to ensure this human inspection occurs. Known as modern code review, this approach is based on tools, such as Gerrit, that help the developers in the reviewing process. As part of this approach, developers are often presented with a list of open code reviews requiring attention; it is left to the developer to find a suitable review on which to work on from a long list of reviews. We present an investigation of two algorithms that recommend an ordering of the list of open reviews based on properties of the reviews. We use a simulation study over the JGit project from the Eclipse Foundation to show that an algorithm based on ordering reviews from least lines of codes changed in the code review to most lines of code out performs other algorithms. This algorithm shows promise for eliminating stagnation of reviews and optimizing the average duration reviews are open.

References

[1]

A. Bacchelli and C. Bird. Expectations, Outcomes, and Challenges of Modern Code Review. In Proc. of the Int’l Conf. on Soft. Eng., ICSE ’13, pages 712–721. IEEE Press, 2013.

Digital Library

Google Scholar

[2]

M. E. Fagan. Design and Code Inspections to Reduce Errors in Program Development. IBM Syst. J., 15(3):182–211, 1976.

Digital Library

Google Scholar

[3]

A. N. Meyer, T. Fritz, G. C. Murphy, and T. Zimmermann. Software Developers’ Perceptions of Productivity. In Proc. of the 22nd ACM SIGSOFT Int’l Symp. on Found. of Soft. Eng., pages 19–29, 2014.

Digital Library

Google Scholar

[4]

P. C. Rigby and C. Bird. Convergent Contemporary Software Peer Review Practices. In Proc. of the 9th Joint Meeting on Found.of Soft. Eng., pages 202–212, 2013.

Digital Library

Google Scholar

[5]

P. C. Rigby, D. M. German, and M.-A. Storey. Open Source Software Peer Review Practices: A Case Study of the Apache Server. In Proc. of the 30th Int’l Conf. on Soft. Eng., pages 541–550, 2008.

Digital Library

Google Scholar

Cited By

View all

Badampudi DUnterkalmsteiner MBritto R(2023)Modern Code Reviews—Survey of Literature and PracticeACM Transactions on Software Engineering and Methodology10.1145/358500432:4(1-61)Online publication date: 26-May-2023
https://dl.acm.org/doi/10.1145/3585004
Badampudi DBritto RUnterkalmsteiner MAli SGarousi V(2019)Modern code reviews - Preliminary results of a systematic mapping studyProceedings of the 23rd International Conference on Evaluation and Assessment in Software Engineering10.1145/3319008.3319354(340-345)Online publication date: 15-Apr-2019
https://dl.acm.org/doi/10.1145/3319008.3319354
dos Santos ENunes I(2018)Investigating the effectiveness of peer code review in distributed software development based on objective and subjective dataJournal of Software Engineering Research and Development10.1186/s40411-018-0058-06:1Online publication date: 26-Oct-2018
https://doi.org/10.1186/s40411-018-0058-0
Show More Cited By

Index Terms

Removing stagnation from modern code review
1. Social and professional topics
  1. Professional topics
    1. Management of computing and information systems
      1. Software management
2. Software and its engineering

Recommendations

Code smells detection via modern code review: a study of the OpenStack and Qt communities
Abstract
Code review plays an important role in software quality control. A typical review process involves a careful check of a piece of code in an attempt to detect and locate defects and other quality issues/violations. One type of issue that may impact ...
Social aspects of a continuous inspection platform for software source code
CHASE '08: Proceedings of the 2008 international workshop on Cooperative and human aspects of software engineering

Determining responsibility for a piece of source code is difficult when software is being developed collaboratively with weak code ownership. Nonetheless, a major factor for preventing "cowboy coding" and careless development of code is liability. We ...
A method for detecting faulty code violating implicit coding rules
IWPSE '02: Proceedings of the International Workshop on Principles of Software Evolution

In the field of legacy software maintenance, there unexpectedly arise a large number of implicit coding rules --- which are seldom written down in specification documents or design documents --- as software becomes more complicated than it used be. ...

Comments

Please enable JavaScript to view thecomments powered by Disqus.

Information & Contributors

Information

Published In

SPLASH Companion 2016: Companion Proceedings of the 2016 ACM SIGPLAN International Conference on Systems, Programming, Languages and Applications: Software for Humanity

October 2016

72 pages

ISBN:9781450344371

DOI:10.1145/2984043

General Chair:
Eelco Visser
Delft University of Technology, Netherlands

Permission to make digital or hard copies of all or part of this work for personal or classroom use is granted without fee provided that copies are not made or distributed for profit or commercial advantage and that copies bear this notice and the full citation on the first page. Copyrights for components of this work owned by others than the author(s) must be honored. Abstracting with credit is permitted. To copy otherwise, or republish, to post on servers or to redistribute to lists, requires prior specific permission and/or a fee. Request permissions from [email protected].

In-Cooperation

SIGAda: ACM Special Interest Group on Ada Programming Language

Publisher

Association for Computing Machinery

New York, NY, United States

Publication History

Published: 20 October 2016

Permissions

Request permissions for this article.

Request Permissions

Check for updates

Author Tags

Qualifiers

Research-article

Funding Sources

Natural Sciences and Engineering Research Council of Canada

Conference

SPLASH '16

Sponsor:

SIGPLAN

SPLASH '16: Conference on Systems, Programming, Languages, and Applications: Software for Humanity

October 30 - November 4, 2016

Amsterdam, Netherlands

Upcoming Conference

Contributors

Other Metrics

View Article Metrics

Bibliometrics & Citations

Bibliometrics

Article Metrics

4
Total Citations
View Citations
224
Total Downloads

Downloads (Last 12 months)1
Downloads (Last 6 weeks)0

Reflects downloads up to 28 Sep 2024

Other Metrics

View Author Metrics

Citations

Cited By

View all

Badampudi DUnterkalmsteiner MBritto R(2023)Modern Code Reviews—Survey of Literature and PracticeACM Transactions on Software Engineering and Methodology10.1145/358500432:4(1-61)Online publication date: 26-May-2023
https://dl.acm.org/doi/10.1145/3585004
Badampudi DBritto RUnterkalmsteiner MAli SGarousi V(2019)Modern code reviews - Preliminary results of a systematic mapping studyProceedings of the 23rd International Conference on Evaluation and Assessment in Software Engineering10.1145/3319008.3319354(340-345)Online publication date: 15-Apr-2019
https://dl.acm.org/doi/10.1145/3319008.3319354
dos Santos ENunes I(2018)Investigating the effectiveness of peer code review in distributed software development based on objective and subjective dataJournal of Software Engineering Research and Development10.1186/s40411-018-0058-06:1Online publication date: 26-Oct-2018
https://doi.org/10.1186/s40411-018-0058-0
dos Santos ENunes IMaldonado JFerrari FKulesza UUchôa Conte T(2017)Investigating the Effectiveness of Peer Code Review in Distributed Software DevelopmentProceedings of the XXXI Brazilian Symposium on Software Engineering10.1145/3131151.3131161(84-93)Online publication date: 20-Sep-2017
https://dl.acm.org/doi/10.1145/3131151.3131161

View Options

Get Access

Login options

Check if you have access through your login credentials or your institution to get full access on this article.

Cited By

Index Terms

Recommendations

Code smells detection via modern code review: a study of the OpenStack and Qt communities

Social aspects of a continuous inspection platform for software source code

A method for detecting faulty code violating implicit coding rules