<html>
  <head>
    <meta content="text/html; charset=windows-1252"
      http-equiv="Content-Type">
  </head>
  <body bgcolor="#FFFFFF" text="#000000">
    <div class="moz-cite-prefix">On 04/24/2015 04:14 AM, Julien Danjou
      wrote:<br>
    </div>
    <blockquote cite="mid:m2wq12uemp.fsf@danjou.info" type="cite">
      <pre wrap="">Hi there,

This is now happening weekly to me now, probably because I write too
many patches touching almost all OpenStack projects once a cycle, and
I'm really tired of that behavior, so PLEASE:

  *Stop sending Code-Review-1 when asking a question in a patch*</pre>
    </blockquote>
    <br>
    Counterpoint:  If you don't, you will not get a response from the
    author.<br>
    <br>
    Too much information, too much churn, but a -1 demands a r response,
    and 0 does not.<br>
    <br>
    Perhaps if we could separately indicate a question that needs to be
    answered from a "do not merge" message,  this would
    change....something like:<br>
    <br>
    <br>
    Q:  "Is Foo the bar?"  sets the question flag.<br>
    <br>
    Me: "No Foo is not the bar."  Indicates that I think I have answered
    the question and clears the flag.<br>
    <br>
    Q then gets notification, and can say yes or no.  If unanswered
    after a day/week, the Flag is automatically cleared.<br>
    <br>
    <br>
    But...that is what -1 is for.  It means: don't merge until my
    question is answered.  If an author could mark that they feel
    they've addressed the -1 without a resubmission, it would take the
    stink off the -1.<br>
    <br>
    <br>
    <blockquote cite="mid:m2wq12uemp.fsf@danjou.info" type="cite">
      <pre wrap="">

_Sometimes_ there are good reasons to set -1 even when asking a
question. For example, when the question is a hint sent to the patch
author so that (s)he improves is commit message, a code comment or a
piece of code.

But most of the time, if you ask a question because there's something
YOU DO NOT KNOW OR UNDERSTAND, do not put a score to a patchset. You
don't know the answer, so you have absolutely no right to evaluate a
patchset with -1. Just don't set a score, it's OK, and wait for the
answer before deciding if the patch is worth [-1..+2].

Thank you for listening, and happy hacking!

</pre>
      <br>
      <fieldset class="mimeAttachmentHeader"></fieldset>
      <br>
      <pre wrap="">__________________________________________________________________________
OpenStack Development Mailing List (not for usage questions)
Unsubscribe: <a class="moz-txt-link-abbreviated" href="mailto:OpenStack-dev-request@lists.openstack.org?subject:unsubscribe">OpenStack-dev-request@lists.openstack.org?subject:unsubscribe</a>
<a class="moz-txt-link-freetext" href="http://lists.openstack.org/cgi-bin/mailman/listinfo/openstack-dev">http://lists.openstack.org/cgi-bin/mailman/listinfo/openstack-dev</a>
</pre>
    </blockquote>
    <br>
  </body>
</html>