[prev in list] [next in list] [prev in thread] [next in thread] 

List:       fedora-directory-devel
Subject:    =?utf-8?q?=5B389-devel=5D?= Re: Please review: 48861 Memberof plugins can update several times the s
From:       thierry bordaz <tbordaz () redhat ! com>
Date:       2016-12-06 13:18:42
Message-ID: 5846BAB2.4070203 () redhat ! com
[Download RAW message or body]

[Attachment #2 (multipart/alternative)]


Thanks William for reviewing the patch.

I fully agree with your remarks and did the changes in

  * https://fedorahosted.org/389/attachment/ticket/48861/0003-Ticket-48861-Memberof-plugins-can-update-several-tim.patch
                
  * https://fedorahosted.org/389/attachment/ticket/48861/0003-Ticket-48861-memberof-plugin-tests-suite.patch



thanks
thierry

On 12/05/2016 11:35 AM, thierry bordaz wrote:
> This is a first patch improving memberof plugins when fixup is done 
> over a graph with multiple paths to the same nodes.
> 
> patch is 
> https://fedorahosted.org/389/attachment/ticket/48861/0002-Ticket-48861-Memberof-plugins-can-update-several-tim.patch
>  
> test suite (port of tet test suite): 
> https://fedorahosted.org/389/attachment/ticket/48861/0002-Ticket-48861-memberof-plugin-tests-suite.patch
>  Compare to the tet tests suite this patch does not contain:
> 
> *  bugs 834053  and 833222 regression tests
> * stress tests
> 
> Design is 
> http://www.port389.org/docs/389ds/design/memberof-scalability.html
> 
> thanks
> thierry
> 
> 
> 
> _______________________________________________
> 389-devel mailing list -- 389-devel@lists.fedoraproject.org
> To unsubscribe send an email to 389-devel-leave@lists.fedoraproject.org


[Attachment #5 (text/html)]

<html>
  <head>
    <meta content="text/html; charset=utf-8" http-equiv="Content-Type">
  </head>
  <body text="#000000" bgcolor="#FFFFFF">
    Thanks William for reviewing the patch.<br>
    <br>
    I fully agree with your remarks and did the changes in <br>
    <ul>
      <li><a class="moz-txt-link-freetext" \
href="https://fedorahosted.org/389/attachment/ticket/48861/0003-Ticket-48861-Memberof- \
plugins-can-update-several-tim.patch">https://fedorahosted.org/389/attachment/ticket/48861/0003-Ticket-48861-Memberof-plugins-can-update-several-tim.patch</a></li>
  <li><a class="moz-txt-link-freetext" \
href="https://fedorahosted.org/389/attachment/ticket/48861/0003-Ticket-48861-memberof- \
plugin-tests-suite.patch">https://fedorahosted.org/389/attachment/ticket/48861/0003-Ticket-48861-memberof-plugin-tests-suite.patch</a></li>
  </ul>
    <br>
    thanks<br>
    thierry<br>
    <br>
    <div class="moz-cite-prefix">On 12/05/2016 11:35 AM, thierry bordaz
      wrote:<br>
    </div>
    <blockquote cite="mid:58454302.8090207@redhat.com" type="cite">
      <meta http-equiv="content-type" content="text/html; charset=utf-8">
      This is a first patch improving memberof plugins when fixup is
      done over a graph with multiple paths to the same nodes.<br>
      <br>
      patch is
      <a moz-do-not-send="true" class="moz-txt-link-freetext"
href="https://fedorahosted.org/389/attachment/ticket/48861/0002-Ticket-48861-Memberof- \
plugins-can-update-several-tim.patch">https://fedorahosted.org/389/attachment/ticket/48861/0002-Ticket-48861-Memberof-plugins-can-update-several-tim.patch</a><br>
  <br>
      test suite (port of tet test suite):
      <a moz-do-not-send="true" class="moz-txt-link-freetext"
href="https://fedorahosted.org/389/attachment/ticket/48861/0002-Ticket-48861-memberof- \
plugin-tests-suite.patch">https://fedorahosted.org/389/attachment/ticket/48861/0002-Ticket-48861-memberof-plugin-tests-suite.patch</a><br>
  Compare to the tet tests suite this patch does not contain:<br>
      <ul>
        <li>  bugs 834053   and 833222 regression tests</li>
        <li>stress tests</li>
      </ul>
      <p>Design is <a moz-do-not-send="true"
          class="moz-txt-link-freetext"
href="http://www.port389.org/docs/389ds/design/memberof-scalability.html">http://www.port389.org/docs/389ds/design/memberof-scalability.html</a><br>
  </p>
      <p>thanks<br>
        thierry<br>
      </p>
      <br>
      <fieldset class="mimeAttachmentHeader"></fieldset>
      <br>
      <pre wrap="">_______________________________________________
389-devel mailing list -- <a class="moz-txt-link-abbreviated" \
href="mailto:389-devel@lists.fedoraproject.org">389-devel@lists.fedoraproject.org</a> \
To unsubscribe send an email to <a class="moz-txt-link-abbreviated" \
href="mailto:389-devel-leave@lists.fedoraproject.org">389-devel-leave@lists.fedoraproject.org</a>
 </pre>
    </blockquote>
    <br>
  </body>
</html>


[Attachment #6 (text/plain)]

_______________________________________________
389-devel mailing list -- 389-devel@lists.fedoraproject.org
To unsubscribe send an email to 389-devel-leave@lists.fedoraproject.org


[prev in list] [next in list] [prev in thread] [next in thread]

Configure | About | News | Add a list | Sponsored by KoreLogic