<div dir="ltr"><br><div class="gmail_extra"><br><div class="gmail_quote">On Sun, Aug 27, 2017 at 3:43 AM, Alex Bradbury via llvm-dev <span dir="ltr"><<a href="mailto:llvm-dev@lists.llvm.org" target="_blank">llvm-dev@lists.llvm.org</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><span class="">On 27 August 2017 at 01:50, Bekket McClane <<a href="mailto:bekket.mcclane@gmail.com">bekket.mcclane@gmail.com</a>> wrote:<br>
> Hi Alex,<br>
><br>
> Thank you very much for raising this crucial issue again.<br>
> And I think the review corner would be a good way to solve it.<br>
><br>
> However, I’m worry about that the traffic of the list would eventually become too high, and looses the ‘highlighter’ feature that it should have<br>
> After all, no one wants the ‘headline of unreviewed patches' to be a 1000+ items list.<br>
<br>
</span>Hi Bekket, thanks for the response.<br>
<br>
I've been following llvm-commits pretty intently for the last 6 weeks<br>
or so. My impression is that the list of patches that are stuck on<br>
reviews and received a 'ping' in that time isn't too huge. I think the<br>
important thing to remember is that the review process is working well<br>
for a large volume of patches, so we're talking about highlighting a<br>
fairly small subset.<br>
<span class=""><br></span></blockquote><div><br></div><div>FWIW: I built and ran a gcc patch tracker back in the day that parsed patches out of the mailing list that were waiting review, identified likely code owners, and built a table, and colored them by status.</div><div><br></div><div>It worked remarkably well to see which areas of the compiler needed more people who could review patches, etc.</div><div>(it was not really meant to shame people :P).</div><div><br></div><div>It ended up getting very heavily used, and it presented relatively hard data.</div><div><br></div><div>So i suspect what you suggest, or some variant may help.</div><div><br></div><div>(It also seems like it would be really easy to  do the above in phab now).</div><div><br></div></div></div></div>