<div dir="ltr"><div>To follow up on Robert's post, Gerrit's query command [1] supports the retrieval of comments for each patchset so maybe we just need to add the '--comments' option when requesting changes [2] ?</div><div><br></div>Since I've posted, I'll just provide my own opinion on this topic. I have no problems with a -1 ask a question vote for any reason but I do think that it's irresponsible to _not_ follow up. So it's the follow up that I would like to see fixed not the restriction on reviews because I feel that it would discourage reviews. <div><br></div><div>[1] <a href="https://review.openstack.org/Documentation/cmd-query.html">https://review.openstack.org/Documentation/cmd-query.html</a></div><div>[2] <a href="https://git.openstack.org/cgit/openstack-infra/reviewstats/tree/reviewstats/utils.py#n170">https://git.openstack.org/cgit/openstack-infra/reviewstats/tree/reviewstats/utils.py#n170</a></div><div><br><div><div class="gmail_extra"><br><div class="gmail_quote">On Fri, Apr 24, 2015 at 9:04 PM, Robert Collins <span dir="ltr"><<a href="mailto:robertc@robertcollins.net" target="_blank">robertc@robertcollins.net</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left-width:1px;border-left-color:rgb(204,204,204);border-left-style:solid;padding-left:1ex"><span class=""><br>
</span>I believe we'll need to switch to the REST JSON API, which gertty uses<br>
and gets comments -> some nontrivial refactoring implied, and I<br>
shudder to think about the performance impact.<br>
<br>
I've wanted to rewrite reviewstats for a while... this might be the trigger...<br>
<span class="im"><br>
-Rob<br>
<br>
<br>
--<br>
Robert Collins <<a href="mailto:rbtcollins@hp.com">rbtcollins@hp.com</a>><br>
Distinguished Technologist<br>
HP Converged Cloud<br>
<br>
</span><div class=""><div class="h5">__________________________________________________________________________<br>
OpenStack Development Mailing List (not for usage questions)<br>
Unsubscribe: <a href="http://OpenStack-dev-request@lists.openstack.org?subject:unsubscribe" target="_blank">OpenStack-dev-request@lists.openstack.org?subject:unsubscribe</a><br>
<a href="http://lists.openstack.org/cgi-bin/mailman/listinfo/openstack-dev" target="_blank">http://lists.openstack.org/cgi-bin/mailman/listinfo/openstack-dev</a><br>
</div></div></blockquote></div><br></div></div></div></div>