Prior work related to this study can be divided into two areas: first, on code review in open source software development;
and second, on the effect of organizational structure on the effectiveness. We now provide main findings for each area.
Rigby and German [13] presented a first study that investigated the code review processes in open source projects.
They compared the code review processes of four open source projects: GCC, Linux, Mozilla, and Apache. They discovered a number of review patterns and performed a quantitative analysis of the review process of the Apache project. Later Rigby et al. [4] analyzed 2,603 patches of the Apache open source system and found that small, independent, complete patches are more likely to be accepted. They found that 44% of submitted patches got accepted compared to 46% in our study. In our study, we differentiate negative and positive reviews and investigate what factors may affect time to acceptance or rejection.
Weissgerber et al. [3] performed data mining on email archives of two open source projects to study patch contributions.
They found that the probability of a patch being accepted is about 40% and that smaller patches have higher chance of
being accepted than larger ones. They also reported that if patches are accepted, they are normally accepted quickly (61% of patches are accepted within three days). Our findings show that 91% of WebKit patches are accepted within 24 hours (ignoring slowest 5% of patches from the analysis).
We have previously studied the code review process of the Mozilla Firefox project, in particular the differences in the patch lifecycles and time taken for each transition for pre- and pos-rapid development models [6]. When analysing Firefox
patch acceptance rate, we did not account for the patch size. In this study we investigated the affect of various factors and
dimensions on the review time and outcome.
Jiang et al. [5] studied the relation of patch characteristics with the probability of patch acceptance and the time taken
for patches to be integrated into the codebase on the example of the Linux kernel. They found that patch acceptance is
affected by the developer experience, patch maturity and priori subsystem churn, while reviewing time is impacted
by submission time, the number of affected subsystems, the number of suggested reviewers and developer experience.
While their patch characteristics do not line up with the factors we studied, we agreed on the same finding that developer
experience correlates with the review time. We also found that for the WebKit project response time is affected by the
reviewer activity, organization, component and patch size.
Most of the related studies on code review perform mining on project’s commit history and thus are not able to reason
about negative feedback and rejection interval. We extracted information from the WebKit’s issue tracking and code review systems providing a more comprehensive view of the code review process.
While we are not aware of a published work on the WebKit case study, Bitergia’s blog provides a general analysis of the
WebKit review process, highlighting trends and summaries of how organizations contribute to the project in terms of both
patch submission and reviewing activity [14]
ก่อนที่การทำงานที่เกี่ยวข้องกับการศึกษาครั้งนี้สามารถแบ่งออกเป็นสองพื้นที่:
ครั้งแรกในการตรวจสอบรหัสในการพัฒนาโอเพนซอร์สซอฟต์แวร์และครั้งที่สองเกี่ยวกับผลกระทบของการจัดโครงสร้างองค์กรในประสิทธิภาพ ตอนนี้เราให้การค้นพบหลักสำหรับแต่ละพื้นที่. Rigby และเยอรมัน [13] นำเสนอการศึกษาครั้งแรกที่ตรวจสอบกระบวนการตรวจสอบรหัสในโครงการมาเปิด. พวกเขาเมื่อเทียบกับกระบวนการตรวจสอบรหัสสี่โครงการมาเปิด: GCC, Linux, Mozilla และอาปาเช่ . พวกเขาค้นพบจำนวนของรูปแบบการดำเนินการตรวจสอบและการวิเคราะห์เชิงปริมาณของกระบวนการตรวจสอบของโครงการอาปาเช่ ต่อมา Rigby et al, [4] วิเคราะห์ 2603 แพทช์ของระบบโอเพนซอร์ส Apache และพบว่าขนาดเล็กที่เป็นอิสระ, แพทช์ที่สมบูรณ์มีแนวโน้มที่จะได้รับการยอมรับ พวกเขาพบว่า 44% ของแพทช์ส่งได้รับการยอมรับเมื่อเทียบกับ 46% ในการศึกษาของเรา ในการศึกษาของเราเราแยกความแตกต่างความคิดเห็นเชิงลบและบวกและตรวจสอบสิ่งที่ปัจจัยที่อาจมีผลต่อเวลาที่จะยอมรับหรือปฏิเสธ. Weissgerber et al, [3] การดำเนินการเกี่ยวกับการทำเหมืองข้อมูลที่เก็บอีเมลของทั้งสองโครงการมาเปิดเพื่อศึกษาผลงานของแพทช์. พวกเขาพบว่าน่าจะเป็นของแพทช์ที่ได้รับการยอมรับเป็นประมาณ 40% และแพทช์ขนาดเล็กมีโอกาสสูงในการได้รับการยอมรับกว่าคนที่มีขนาดใหญ่ พวกเขายังมีรายงานว่าถ้าแพทช์ได้รับการยอมรับพวกเขาได้รับการยอมรับตามปกติได้อย่างรวดเร็ว (61% ของแพทช์ได้รับการยอมรับภายในสามวัน) ผลการวิจัยของเราแสดงให้เห็นว่า 91% ของแพทช์ WebKit ได้รับการยอมรับภายใน 24 ชั่วโมง (ไม่สนใจที่ช้าที่สุด 5% ของแพทช์จากการวิเคราะห์). เรามีการศึกษาก่อนหน้านี้กระบวนการตรวจสอบรหัสของโครงการเซอร์ Mozilla Firefox โดยเฉพาะอย่างยิ่งความแตกต่างในวงจรชีวิตแพทช์ที่และเวลา ดำเนินการสำหรับการเปลี่ยนแปลงสำหรับรุ่นก่อนและการพัฒนาอย่างรวดเร็ว POS แต่ละ [6] เมื่อวิเคราะห์ Firefox อัตราการยอมรับแพทช์ที่เราไม่ได้คิดเป็นขนาดแพทช์ ในการศึกษานี้เราตรวจสอบผลกระทบของปัจจัยต่างๆและขนาดในเวลาการตรวจสอบและผล. เจียง et al, [5] การศึกษาความสัมพันธ์ของลักษณะแพทช์ที่มีความน่าจะเป็นของการยอมรับแพทช์และเวลาที่แพทช์ที่จะรวมเข้าcodebase ในตัวอย่างของเคอร์เนลลินุกซ์ พวกเขาพบว่าได้รับการยอมรับแพทช์จะได้รับผลกระทบจากประสบการณ์การพัฒนาครบกําหนดแพทช์และปั่นระบบย่อยเบื้องต้นในขณะที่การตรวจสอบเวลาที่ได้รับผลกระทบตามเวลาที่ส่งจำนวนของระบบย่อยได้รับผลกระทบจำนวนผู้แสดงความคิดเห็นข้อเสนอแนะและประสบการณ์พัฒนา. ในขณะที่ลักษณะของแพทช์ของพวกเขาไม่สาย ขึ้นกับปัจจัยที่เราศึกษาเราเห็นด้วยกับการค้นพบที่เดียวกับที่นักพัฒนาประสบการณ์ที่มีความสัมพันธ์กับเวลาการตรวจสอบ นอกจากนี้เรายังพบว่าสำหรับโครงการ WebKit เวลาตอบสนองที่ได้รับผลกระทบจากกิจกรรมวิจารณ์องค์กรองค์ประกอบและขนาดแพทช์. ส่วนใหญ่ของการศึกษาที่เกี่ยวข้องกับการตรวจสอบรหัสดำเนินการทำเหมืองแร่ในโครงการกระทำประวัติศาสตร์และดังนั้นจึงไม่สามารถที่จะให้เหตุผลเกี่ยวกับการลบความคิดเห็นและการปฏิเสธช่วงเวลา เราสกัดข้อมูลจากการติดตามปัญหาของ WebKit และระบบการตรวจสอบรหัสการให้มุมมองที่ครอบคลุมมากขึ้นของกระบวนการตรวจสอบรหัส. ในขณะที่เราไม่ได้ตระหนักถึงการทำงานที่เผยแพร่ในกรณีศึกษา WebKit บล็อก Bitergia ให้การวิเคราะห์ทั่วไปของกระบวนการตรวจสอบWebKit, ไฮไลท์แนวโน้มและบทสรุปของวิธีที่องค์กรมีส่วนร่วมในโครงการทั้งในแง่ของการส่งแพทช์และทบทวนกิจกรรม[14]
การแปล กรุณารอสักครู่..
