<html><head><meta http-equiv="Content-Type" content="text/html charset=utf-8"></head><body style="word-wrap: break-word; -webkit-nbsp-mode: space; -webkit-line-break: after-white-space;" class="">Hi<div class=""><br class=""></div><div class="">Copying our mailing list, to make the answer publicly available.</div><div class=""><br class=""></div><div class="">Sounds like an interesting topic, but looking at the one you asked me about it seems to me, you look at the abandoned ones only in a statistical sense, that will not do. I am actually curious, why do you look at abandoned patches, the right question (at least to me) is “why do some patches not get merged”, for LibreOffice the result is the same, but for other projects it is very different.</div><div class=""><br class=""></div><div class="">33697 was submitted and abandoned by myself, because it had some problems…you will see a lot of people submit patches, to get them tested on our build boxes, and if there is a problem they either submit a second patch set or (as in my case) abandon the patch and submit a new one.</div><div class=""><br class=""></div><div class="">I trust you are also aware that TDF (in contrary to most other foundations) actively monitor gerrit. Patches who are untouched for 4 weeks, get a “polite ping”, and 4 weeks later still untouched gets abandoned. This procedure gives a lot more abandoned patches compared to other projects.</div><div class=""><br class=""></div><div class="">If you speak generally you need to look more carefully at the reasons.</div><div class=""><br class=""></div><div class="">1) Submitter abandons patch. This is quite natural as described above.</div><div class=""><br class=""></div><div class="">2) People with committer status, typically submit a patch and forget it, so the system catches it and they typically abandon it, when they see the “polite ping”.</div><div class=""><br class=""></div><div class="">3) New people submit a patch (some without having compiled the code) receive a review and decide not to continue. This is only true for first time patches.</div><div class=""><br class=""></div><div class="">4) New people do a good job of making their patch work, but get review comments e.g. due to the user experience not being correct, and stop working</div><div class=""><br class=""></div><div class="">Again compared to other projects, we ensure that all patches get reviewed, so no patch is abandoned due to lack of review (something I see happening in another Foundation I work with).</div><div class=""><br class=""></div><div class="">Have fun with your study, and please share the outcome on our dev list, and I think it will be interesting reading.</div><div class=""><br class=""></div><div class="">rgds</div><div class="">jan I.</div><div class=""><br class=""></div><div class=""><br class=""><div><blockquote type="cite" class=""><div class="">On 10 Mar 2017, at 07:00, <a href="mailto:wqyy@zju.edu.cn" class="">wqyy@zju.edu.cn</a> wrote:</div><br class="Apple-interchange-newline"><div class=""><p class="MsoNormal" style="mso-margin-top-alt:auto;mso-margin-bottom-alt:auto">Dear jan, </p> Sorry to bother you!<br class=""> <br class=""> I am a Ph.D student in Zhejiang University, China. Currently, we are doing a research study on identifying root causes of abandoned code reviews. As we noticed, code reviews are abandoned for various reasons, e.g., build failed, wrong branch, duplicate patch, and so on. <br class=""> <br class=""> Since you are an expert in libreoffice, and I'm writing to ask you the reason why the following code review was abandoned (<a href="https://gerrit.libreoffice.org/#/c/33697/" target="_blank" class="">https://gerrit.libreoffice.org/#/c/33697/</a>). Also, it would be great if you can tell us some more reasons why a code review is abandoned:-) I would be really grateful if you could reply me.<br class=""> <br class=""> Please kindly advice.<br class=""> <br class=""> Thank you very much for your help and happy holidays!<br class=""> <br class=""> Yours sincerely,<br class=""> <br class=""> Qingye Wang</div></blockquote></div><br class=""></div></body></html>