<div dir="ltr">Hi,<br><br><div class="gmail_quote"><div dir="ltr">On Wed, Jun 1, 2016 at 1:06 PM Mark Abraham &lt;<a href="mailto:mark.j.abraham@gmail.com">mark.j.abraham@gmail.com</a>&gt; wrote:<br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div dir="ltr"><div>So I&#39;ve fixed them, and after two months Teemu found time to look at the straightforward fix + tests. After a further month, nobody else has had time to look. Meanwhile, it isn&#39;t worth me continuing development to support such a feature until I see that somebody cares about it.</div></div></blockquote><div><br></div><div>I agree that the delay in getting things reviewed is sometimes very long. But the above is a bit one-sided. It is true that it took me two months to find time to review stuff (because of various reasons, including other Gromacs work and general lack of free time), and there&#39;s not much more to say about that. But once I did provide comments, the next month the patch just sat there in Gerrit, waiting for some of my comments to be addressed. It might very well be that no one else looked at it, but if I had not been the one to post comments and had looked at it, I would have just ignored it because there were still unaddressed comments from someone else. After the findings were addressed, it took all of a four days to get it merged.<br></div><div><br></div><div>The responsiveness in reviewing stuff goes both ways. I have currently some 75 changes in the &quot;incoming reviews&quot; category, which is something like 50% of all open changes in Gerrit! And quite a few of those are really old, where either the change doesn&#39;t even pass Jenkins, where there have been comments from reviewers and then no action whatsoever afterwards, or that has been long since been obsoleted by some other change. This is a huge demotivating factor for reviewing stuff, not to mention that it is a waste of time scanning through the same list over and over again to see whether there would be a change somewhere in that 50+ changes (or in the 150+ open changes) that could be usefully reviewed...</div><div><br></div><div>Best regards,</div><div>Teemu</div></div></div>